Skip to content

Update regex for parsing curl.se Mozilla certs header date#907

Merged
thejcfactor merged 2 commits intocouchbase:mainfrom
thejcfactor:curl_mozilla_cert_header_update
Mar 19, 2026
Merged

Update regex for parsing curl.se Mozilla certs header date#907
thejcfactor merged 2 commits intocouchbase:mainfrom
thejcfactor:curl_mozilla_cert_header_update

Conversation

@thejcfactor
Copy link
Copy Markdown
Contributor

No description provided.

@thejcfactor thejcfactor requested a review from avsej March 19, 2026 18:46
@avsej avsej requested a review from Copilot March 19, 2026 18:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the CMake logic that extracts the “as of / last updated” date from the curl.se Mozilla CA bundle header when embedding certificates during the build.

Changes:

  • Broaden the header regex to match both “as of” and “last updated on”.
  • Update the capture group used to populate COUCHBASE_CXX_CLIENT_MOZILLA_CA_BUNDLE_DATE.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cmake/OpenSSL.cmake Outdated
@mikereiche
Copy link
Copy Markdown

Looks good.

Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@thejcfactor thejcfactor merged commit e4e9772 into couchbase:main Mar 19, 2026
51 of 57 checks passed
@thejcfactor thejcfactor deleted the curl_mozilla_cert_header_update branch March 19, 2026 22:38
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