RE: [DISCUSS] PIP-279: Reformat property in generateResponseWithEntry

[Haiting Jiang]

Hi mingze,

Good catch.
I think we should apply this fix to the released versions,
since this PIP is kind of protocol change and only applies to new version.

Thanks,
Haiting

On 2023/06/29 06:07:18 韩明泽 wrote:
> If do not upgrade the version, you can add the following parameters in
> client to solve this problem.
> 
> -Dorg.asynchttpclient.validateResponseHeaders=false
> -Dorg.apache.pulsar.shade.org.asynchttpclient.validateResponseHeaders=false
> 

RE: [DISCUSS] PIP-279: Reformat property in generateResponseWithEntry

[韩明泽]

If do not upgrade the version, you can add the following parameters in
client to solve this problem.

-Dorg.asynchttpclient.validateResponseHeaders=false
-Dorg.apache.pulsar.shade.org.asynchttpclient.validateResponseHeaders=false

Re: [DISCUSS] PIP-279: Reformat property in generateResponseWithEntry

[Shiji Lu]

the DISCUSS is passed,I have send the VOTE for this PIP
https://lists.apache.org/thread/g354684m9h495o3p0kmzb7fh7vfxhddx


On 2023/06/21 03:22:52 steven lu wrote:
> # Motivation
> 
> reformat property,for a http header name cannot contain the following
> prohibited characters: =,;: \t\r\n\v\f
> 
> for example:
> {"city=shanghai":"tag"}
> when we run `bin/pulsar-admin topics get-message-by-id `, it will
> throw exception, the exception is:
> `Reason: java.util.concurrent.CompletionException:
> org.apache.pulsar.client.admin.internal.http.AsyncHttpConnector$RetryException:
> Could not complete the operation. Number of retries has been
> exhausted. Failed reason: a header name cannot contain the following
> prohibited characters: =,;: \t\r\n\v\f: =`
> 
>  src="https://github.com/StevenLuMT/pulsar/assets/42990025/973d95b9-4ac2-4977-b160-162c4b53a613;>
> 
> # High Level Design
> 
> In master branch,
> in an http 
> request:getMessageById("/{tenant}/{namespace}/{topic}/ledger/{ledgerId}/entry/{entryId}"),
> replace `"X-Pulsar-PROPERTY-" + msgProperties.getKey()` with
> `"X-Pulsar-PROPERTY"`
> 
> After release-3.1.0, this feature begins to take effect.
> 

Re: [DISCUSS] PIP-279: Reformat property in generateResponseWithEntry

[houxiaoyu]

+1 non-binding

Xiaoyu Hou

Enrico Olivelli  于2023年6月26日周一 14:57写道：

> +1 (binding)
>
> Enrico
>
> Il giorno lun 26 giu 2023 alle ore 07:53 guo jiwei
>  ha scritto:
> >
> > +1 (binding)
> >
> > This is a bug and we need this fix.
> >
> >
> >
> >
> > Regards
> > Jiwei Guo (Tboy)
> >
> >
> > On Wed, Jun 21, 2023 at 11:23 AM steven lu 
> wrote:
> >
> > > # Motivation
> > >
> > > reformat property,for a http header name cannot contain the following
> > > prohibited characters: =,;: \t\r\n\v\f
> > >
> > > for example:
> > > {"city=shanghai":"tag"}
> > > when we run `bin/pulsar-admin topics get-message-by-id `, it will
> > > throw exception, the exception is:
> > > `Reason: java.util.concurrent.CompletionException:
> > >
> > >
> org.apache.pulsar.client.admin.internal.http.AsyncHttpConnector$RetryException:
> > > Could not complete the operation. Number of retries has been
> > > exhausted. Failed reason: a header name cannot contain the following
> > > prohibited characters: =,;: \t\r\n\v\f: =`
> > >
> > >  > > src="
> > >
> https://github.com/StevenLuMT/pulsar/assets/42990025/973d95b9-4ac2-4977-b160-162c4b53a613
> > > ">
> > >
> > > # High Level Design
> > >
> > > In master branch,
> > > in an http
> > >
> request:getMessageById("/{tenant}/{namespace}/{topic}/ledger/{ledgerId}/entry/{entryId}"),
> > > replace `"X-Pulsar-PROPERTY-" + msgProperties.getKey()` with
> > > `"X-Pulsar-PROPERTY"`
> > >
> > > After release-3.1.0, this feature begins to take effect.
> > >
>

Re: [DISCUSS] PIP-279: Reformat property in generateResponseWithEntry

[Enrico Olivelli]

+1 (binding)

Enrico

Il giorno lun 26 giu 2023 alle ore 07:53 guo jiwei
 ha scritto:
>
> +1 (binding)
>
> This is a bug and we need this fix.
>
>
>
>
> Regards
> Jiwei Guo (Tboy)
>
>
> On Wed, Jun 21, 2023 at 11:23 AM steven lu  wrote:
>
> > # Motivation
> >
> > reformat property,for a http header name cannot contain the following
> > prohibited characters: =,;: \t\r\n\v\f
> >
> > for example:
> > {"city=shanghai":"tag"}
> > when we run `bin/pulsar-admin topics get-message-by-id `, it will
> > throw exception, the exception is:
> > `Reason: java.util.concurrent.CompletionException:
> >
> > org.apache.pulsar.client.admin.internal.http.AsyncHttpConnector$RetryException:
> > Could not complete the operation. Number of retries has been
> > exhausted. Failed reason: a header name cannot contain the following
> > prohibited characters: =,;: \t\r\n\v\f: =`
> >
> >  > src="
> > https://github.com/StevenLuMT/pulsar/assets/42990025/973d95b9-4ac2-4977-b160-162c4b53a613
> > ">
> >
> > # High Level Design
> >
> > In master branch,
> > in an http
> > request:getMessageById("/{tenant}/{namespace}/{topic}/ledger/{ledgerId}/entry/{entryId}"),
> > replace `"X-Pulsar-PROPERTY-" + msgProperties.getKey()` with
> > `"X-Pulsar-PROPERTY"`
> >
> > After release-3.1.0, this feature begins to take effect.
> >

Re: [DISCUSS] PIP-279: Reformat property in generateResponseWithEntry

[guo jiwei]

+1 (binding)

This is a bug and we need this fix.




Regards
Jiwei Guo (Tboy)


On Wed, Jun 21, 2023 at 11:23 AM steven lu  wrote:

> # Motivation
>
> reformat property,for a http header name cannot contain the following
> prohibited characters: =,;: \t\r\n\v\f
>
> for example:
> {"city=shanghai":"tag"}
> when we run `bin/pulsar-admin topics get-message-by-id `, it will
> throw exception, the exception is:
> `Reason: java.util.concurrent.CompletionException:
>
> org.apache.pulsar.client.admin.internal.http.AsyncHttpConnector$RetryException:
> Could not complete the operation. Number of retries has been
> exhausted. Failed reason: a header name cannot contain the following
> prohibited characters: =,;: \t\r\n\v\f: =`
>
>  src="
> https://github.com/StevenLuMT/pulsar/assets/42990025/973d95b9-4ac2-4977-b160-162c4b53a613
> ">
>
> # High Level Design
>
> In master branch,
> in an http
> request:getMessageById("/{tenant}/{namespace}/{topic}/ledger/{ledgerId}/entry/{entryId}"),
> replace `"X-Pulsar-PROPERTY-" + msgProperties.getKey()` with
> `"X-Pulsar-PROPERTY"`
>
> After release-3.1.0, this feature begins to take effect.
>

[DISCUSS] PIP-279: Reformat property in generateResponseWithEntry

[steven lu]

# Motivation

reformat property,for a http header name cannot contain the following
prohibited characters: =,;: \t\r\n\v\f

for example:
{"city=shanghai":"tag"}
when we run `bin/pulsar-admin topics get-message-by-id `, it will
throw exception, the exception is:
`Reason: java.util.concurrent.CompletionException:
org.apache.pulsar.client.admin.internal.http.AsyncHttpConnector$RetryException:
Could not complete the operation. Number of retries has been
exhausted. Failed reason: a header name cannot contain the following
prohibited characters: =,;: \t\r\n\v\f: =`

https://github.com/StevenLuMT/pulsar/assets/42990025/973d95b9-4ac2-4977-b160-162c4b53a613;>

# High Level Design

In master branch,
in an http 
request:getMessageById("/{tenant}/{namespace}/{topic}/ledger/{ledgerId}/entry/{entryId}"),
replace `"X-Pulsar-PROPERTY-" + msgProperties.getKey()` with
`"X-Pulsar-PROPERTY"`

After release-3.1.0, this feature begins to take effect.