Skip to content

Conversation

@onobc
Copy link
Contributor

@onobc onobc commented Feb 3, 2024

This commit removes the use of the Pulsar ObjectMapperFactory when converting the authentication config props map to a JSON string. The Pulsar factory operates on a shaded returned value of Jackson ObjectMapper which may not exist when users are using the non-shaded version of the Pulsar client lib.

See spring-projects/spring-pulsar#562

This commit removes the use of the Pulsar ObjectMapperFactory when
converting the authentication config props map to a JSON string. The
Pulsar factory operates on a shaded returned value of Jackson
ObjectMapper which may not exist when users are using the
non-shaded version of the Pulsar client lib.

See spring-projects/spring-pulsar#562
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 3, 2024
@wilkinsona wilkinsona changed the title Remove use of Pulsar ObjectMapperFactory NoSuchMethod error when using the non-shaded Pulsar client and configuring authentications parameters Feb 4, 2024
@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 4, 2024
@wilkinsona wilkinsona added this to the 3.2.x milestone Feb 4, 2024
@mhalbritter mhalbritter self-assigned this Feb 6, 2024
mhalbritter pushed a commit that referenced this pull request Feb 6, 2024
This commit removes the use of the Pulsar ObjectMapperFactory when
converting the authentication config props map to a JSON string. The
Pulsar factory operates on a shaded returned value of Jackson
ObjectMapper which may not exist when users are using the
non-shaded version of the Pulsar client lib.

See spring-projects/spring-pulsar#562

See gh-39389
@mhalbritter
Copy link
Contributor

Thanks!

@mhalbritter mhalbritter modified the milestones: 3.2.x, 3.2.3 Feb 6, 2024
@onobc onobc deleted the fix-pulsar-auth-props-unshaded-client-fix branch May 28, 2024 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants