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

Infinispan Configuration enchancements #30065

Merged

Conversation

karesti
Copy link
Member

@karesti karesti commented Dec 23, 2022

  • Deprecate serverList for hosts (consistency with other extensions)
  • Deprecate authUsername for username
  • Deprecate authPassowrd for password
  • Provide uri connection configuration
  • Provide ssl provider, protocol and ciphers closes Support bouncyCastle libs + Infinispan + FIPS #25136

@quarkus-bot

This comment has been minimized.

@karesti karesti force-pushed the deprecate-service-list-in-favor-of-url branch from 6feec51 to 48066a9 Compare January 2, 2023 15:45
@karesti karesti requested a review from geoand January 2, 2023 15:46
@quarkus-bot

This comment has been minimized.

@karesti karesti force-pushed the deprecate-service-list-in-favor-of-url branch from 48066a9 to f02ae9d Compare January 3, 2023 09:34
@quarkus-bot

This comment has been minimized.

@karesti karesti force-pushed the deprecate-service-list-in-favor-of-url branch 2 times, most recently from 649550a to 98c5b5d Compare January 3, 2023 16:39
@quarkus-bot

This comment has been minimized.

@karesti
Copy link
Member Author

karesti commented Jan 4, 2023

@geoand I fixed the javadoc issue I missed

* Deprecate serverList for hosts (consistency with other extensions)
* Deprecate authUsername for username
* Deprecate authPassowrd for password
* Provide uri connection configuration
* Provide ssl provider, protocol and ciphers
@karesti karesti force-pushed the deprecate-service-list-in-favor-of-url branch from 98c5b5d to 00dd66d Compare January 4, 2023 09:13
Copy link
Contributor

@geoand geoand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@geoand geoand added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jan 4, 2023
@karesti
Copy link
Member Author

karesti commented Jan 4, 2023

is there anything else to be validated at this point @geoand ?

@geoand geoand merged commit 8bfeb9d into quarkusio:main Jan 5, 2023
@quarkus-bot quarkus-bot bot added this to the 2.16 - main milestone Jan 5, 2023
@quarkus-bot quarkus-bot bot added kind/bugfix and removed triage/waiting-for-ci Ready to merge when CI successfully finishes labels Jan 5, 2023
@karesti karesti deleted the deprecate-service-list-in-favor-of-url branch January 5, 2023 08:54
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.

Support bouncyCastle libs + Infinispan + FIPS
2 participants