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.

Sign up for GitHub

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

Jump to bottom

WFLY-12662 BouncyCastle update from 1.60 to 1.64 #12709

Merged
merged 1 commit into from Oct 17, 2019
Merged

WFLY-12662 BouncyCastle update from 1.60 to 1.64 #12709

merged 1 commit into from Oct 17, 2019

Conversation

rsvoboda
Copy link
Contributor

BouncyCastle update from 1.60 to 1.64
https://issues.jboss.org/browse/WFLY-12662

Please make sure your PR meets the following requirements:

  • Pull Request title is properly formatted: [WFLY-XYZ] Subject or WFLY-XYZ Subject
  • Pull Request contains link to the JIRA issue(s)
  • Pull Request contains description of the issue(s)
  • Pull Request does not include fixes for issues other than the main ticket
  • Attached commits represent units of work and are properly formatted

@bstansberry
Copy link
Contributor

@darranl or @fjuma Please review.

@wildfly-ci wildfly-ci added the deps-ok Dependencies have been checked, and there are no significant changes label Oct 14, 2019
@darranl
Copy link
Contributor

darranl commented Oct 15, 2019

I believe Rest Easy may be the bigger consumer of BouncyCastle - I generally don't have an issue for small incremental updates if CI passes but they may want to double check as well.

@bstansberry
Copy link
Contributor

@ronsigal Any input?

bstansberry
bstansberry approved these changes Oct 16, 2019
View reviewed changes
Copy link
Contributor

@bstansberry bstansberry left a comment

Choose a reason for hiding this comment

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

I got input from Ron and Alessio that this looked ok from the RESTEasy point of view.

@bstansberry bstansberry added the ready-for-merge Only for use by those with merge permissions! label Oct 16, 2019
@bstansberry bstansberry mentioned this pull request Oct 16, 2019
Closed
@bstansberry bstansberry merged commit 8b35dda into wildfly:master Oct 17, 2019
@bstansberry bstansberry mentioned this pull request Oct 28, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bstansberry bstansberry bstansberry approved these changes

Assignees
No one assigned
Labels
deps-ok Dependencies have been checked, and there are no significant changes ready-for-merge Only for use by those with merge permissions!
Projects
None yet
Milestone
No milestone
4 participants
@rsvoboda @bstansberry @darranl @wildfly-ci