Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[python functions] fix tls_validate_hostname is not supported in python functions runtime #11087

Merged

Conversation

freeznet
Copy link
Contributor

@freeznet freeznet commented Jun 25, 2021

Motivation

Pulsar's python client support define TLS parameters, like tls_validate_hostname, but in python functions runtime, it is not been set properly.

Modifications

  • get hostname_verification_enabled from python instance arguments
  • pass hostname_verification_enabled to pulsar client

Verifying this change

  • Make sure that the change passes the CI checks.

@sijie
Copy link
Member

sijie commented Jul 14, 2021

@nlu90 Can you review this?

@sijie sijie merged commit a14f1c5 into apache:master Jul 20, 2021
lhotari added a commit to lhotari/pulsar that referenced this pull request Jul 21, 2021
Technoboy- pushed a commit to Technoboy-/pulsar that referenced this pull request Jul 22, 2021
…on functions runtime (apache#11087)

### Motivation

Pulsar's python client support define TLS parameters, like `tls_validate_hostname`, but in python functions runtime, it is not been set properly. 

### Modifications

- get `hostname_verification_enabled` from python instance arguments
- pass `hostname_verification_enabled` to pulsar client

### Verifying this change

- [x] Make sure that the change passes the CI checks.
codelipenghui pushed a commit that referenced this pull request Jul 23, 2021
…on functions runtime (#11087)

### Motivation

Pulsar's python client support define TLS parameters, like `tls_validate_hostname`, but in python functions runtime, it is not been set properly. 

### Modifications

- get `hostname_verification_enabled` from python instance arguments
- pass `hostname_verification_enabled` to pulsar client

### Verifying this change

- [x] Make sure that the change passes the CI checks.

(cherry picked from commit a14f1c5)
@codelipenghui codelipenghui added the cherry-picked/branch-2.8 Archived: 2.8 is end of life label Jul 23, 2021
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
…on functions runtime (apache#11087)

### Motivation

Pulsar's python client support define TLS parameters, like `tls_validate_hostname`, but in python functions runtime, it is not been set properly. 

### Modifications

- get `hostname_verification_enabled` from python instance arguments
- pass `hostname_verification_enabled` to pulsar client

### Verifying this change

- [x] Make sure that the change passes the CI checks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants