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

fix #6834: add pulsar-client-messagecrypto-bc into pulsar-client dependency to avoid method not found #7447

Merged
merged 1 commit into from
Jul 5, 2020

Conversation

jiazhai
Copy link
Member

@jiazhai jiazhai commented Jul 3, 2020

Fixes #6834

Motivation

MessageCryptoBc implements the interface of MessageCrypto. while pulsar-client did a shade toByteBuf, and caused the method parameter changed from org.apache.pulsar.shade.io.netty.buffer.ByteBuf to io.netty.buffer.ByteBuf and will cause error of:

Caused by: java.lang.NoSuchMethodError: 'org.apache.pulsar.shade.io.netty.buffer.ByteBuf org.apache.pulsar.client.api.MessageCrypto.encrypt(java.util.Set, org.apache.pulsar.client.api.CryptoKeyReader, java.util.function.Supplier, org.apache.pulsar.shade.io.netty.buffer.ByteBuf)

Modifications

add pulsar-client-messagecrypto-bc into pulsar-client dependency to avoid method-not-found issue.

@jiazhai
Copy link
Member Author

jiazhai commented Jul 4, 2020

/pulsarbot run-failure-checks

1 similar comment
@jiazhai
Copy link
Member Author

jiazhai commented Jul 4, 2020

/pulsarbot run-failure-checks

@codelipenghui codelipenghui added this to the 2.7.0 milestone Jul 5, 2020
@jiazhai jiazhai merged commit d83133a into apache:master Jul 5, 2020
codelipenghui pushed a commit to streamnative/pulsar-archived that referenced this pull request Jul 14, 2020
…7447)

Fixes apache#6834

### Motivation

`MessageCryptoBc` implements the interface of `MessageCrypto`. while `pulsar-client` did a shade to`ByteBuf`, and caused the method parameter changed from `org.apache.pulsar.shade.io.netty.buffer.ByteBuf` to `io.netty.buffer.ByteBuf` and will cause error of:
```
Caused by: java.lang.NoSuchMethodError: 'org.apache.pulsar.shade.io.netty.buffer.ByteBuf org.apache.pulsar.client.api.MessageCrypto.encrypt(java.util.Set, org.apache.pulsar.client.api.CryptoKeyReader, java.util.function.Supplier, org.apache.pulsar.shade.io.netty.buffer.ByteBuf)
```

### Modifications

add pulsar-client-messagecrypto-bc into pulsar-client dependency to avoid method-not-found issue.

(cherry picked from commit d83133a)
wolfstudy pushed a commit that referenced this pull request Jul 29, 2020
Fixes #6834

### Motivation

`MessageCryptoBc` implements the interface of `MessageCrypto`. while `pulsar-client` did a shade to`ByteBuf`, and caused the method parameter changed from `org.apache.pulsar.shade.io.netty.buffer.ByteBuf` to `io.netty.buffer.ByteBuf` and will cause error of:
```
Caused by: java.lang.NoSuchMethodError: 'org.apache.pulsar.shade.io.netty.buffer.ByteBuf org.apache.pulsar.client.api.MessageCrypto.encrypt(java.util.Set, org.apache.pulsar.client.api.CryptoKeyReader, java.util.function.Supplier, org.apache.pulsar.shade.io.netty.buffer.ByteBuf)
```

### Modifications

add pulsar-client-messagecrypto-bc into pulsar-client dependency to avoid method-not-found issue.

(cherry picked from commit d83133a)
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Aug 24, 2020
…7447)

Fixes apache#6834

### Motivation

`MessageCryptoBc` implements the interface of `MessageCrypto`. while `pulsar-client` did a shade to`ByteBuf`, and caused the method parameter changed from `org.apache.pulsar.shade.io.netty.buffer.ByteBuf` to `io.netty.buffer.ByteBuf` and will cause error of:
```
Caused by: java.lang.NoSuchMethodError: 'org.apache.pulsar.shade.io.netty.buffer.ByteBuf org.apache.pulsar.client.api.MessageCrypto.encrypt(java.util.Set, org.apache.pulsar.client.api.CryptoKeyReader, java.util.function.Supplier, org.apache.pulsar.shade.io.netty.buffer.ByteBuf)
```

### Modifications

add pulsar-client-messagecrypto-bc into pulsar-client dependency to avoid method-not-found issue.
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.

Apache pulsar Encryption failed using ecdsa
2 participants