Skip to content

nginx-1.28.0 aws-lc-nginx.patch#2373

Merged
justsmth merged 1 commit intoaws:mainfrom
robvanoostenrijk:nginx-1.28.0-patch
Apr 29, 2025
Merged

nginx-1.28.0 aws-lc-nginx.patch#2373
justsmth merged 1 commit intoaws:mainfrom
robvanoostenrijk:nginx-1.28.0-patch

Conversation

@robvanoostenrijk
Copy link
Copy Markdown
Contributor

@robvanoostenrijk robvanoostenrijk commented Apr 27, 2025

Description of changes:

Updated integrations aws-lc-nginx.patch against nginx 1.28.0

Testing:

Successful build of nginx 1.28.0 against AWS-LC 1.50.0

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

@robvanoostenrijk robvanoostenrijk requested a review from a team as a code owner April 27, 2025 04:28
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.77%. Comparing base (6985ddf) to head (85532af).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2373      +/-   ##
==========================================
+ Coverage   78.75%   78.77%   +0.01%     
==========================================
  Files         620      620              
  Lines      107958   107958              
  Branches    15331    15331              
==========================================
+ Hits        85022    85039      +17     
+ Misses      22280    22262      -18     
- Partials      656      657       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@justsmth
Copy link
Copy Markdown
Contributor

Hello! We require contributions to align with our open source licensing. Our CI enforces this by expecting every PR description to contain the following text:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

If you agree, would you update the PR description with the verbiage above?

@robvanoostenrijk
Copy link
Copy Markdown
Contributor Author

License statement added

@justsmth justsmth merged commit 3990a83 into aws:main Apr 29, 2025
117 of 120 checks passed
@justsmth justsmth mentioned this pull request Apr 30, 2025
justsmth added a commit that referenced this pull request Apr 30, 2025
### Description of changes: 
Bump version to 1.50.1. (Needed so that `aws-lc-sys` can to pick up
latest s2n-bignum changes.)

### Call-outs:
#### What's Changed
* Fix GCC 4.8 docker img; Also w/ GCC 7.5 by @justsmth in
#2344
* Fix LibRdKafka CI by @smittals2 in
#2372
* Expand .clang-tidy configuration by @justsmth in
#2356
* nginx-1.28.0 aws-lc-nginx.patch by @robvanoostenrijk in
#2373
* s2n bignum import method change by @torben-hansen in
#2324
* Fix a theoretical overflow in BIO_printf by @justsmth in
#2369
* Fix tpm2-tss integration test by @justsmth in
#2370


By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license and the ISC license.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants