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

Bump BoringSSL submodule version #643

Merged
merged 1 commit into from
May 25, 2020
Merged

Bump BoringSSL submodule version #643

merged 1 commit into from
May 25, 2020

Conversation

ilammy
Copy link
Collaborator

@ilammy ilammy commented May 25, 2020

Pull latest updates for BoringSSL. Since BoringSSL does not have release schedule or versions, we just pull the latest stuff from master branch.

This affects mostly AndroidThemis and WasmThemis since they are using BoringSSL by default. Other platforms normally rely on OpenSSL.

Checklist

  • Change is covered by automated tests
  • Changelog is updated

@ilammy
Bump BoringSSL submodule version
63ccfaa 
Pull latest updates for BoringSSL. Since BoringSSL does not have release
schedule or versions, we just pull the latest stuff from master branch.

This affects mostly AndroidThemis and WasmThemis since they are using
BoringSSL by default. Other platforms normally rely on OpenSSL.
@ilammy ilammy added core Themis Core written in C, its packages W-JavaThemis ☕ Wrapper: Java, Java and Kotlin API C-BoringSSL Crypto provider: BoringSSL W-WasmThemis 🌐 Wrapper: WasmThemis, JavaScript API, npm packages labels May 25, 2020
@vixentael
Copy link
Contributor

Let's hope that tests are passing and that we have enough tests to note sudden changes, hehe

@ilammy ilammy merged commit c075ba8 into cossacklabs:master May 25, 2020
@ilammy ilammy deleted the bump-boringssl branch May 25, 2020 12:16
@ilammy ilammy mentioned this pull request May 26, 2020
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vixentael vixentael vixentael approved these changes

@Lagovas Lagovas Awaiting requested review from Lagovas Lagovas is a code owner

Assignees
No one assigned
Labels
C-BoringSSL Crypto provider: BoringSSL core Themis Core written in C, its packages W-JavaThemis ☕ Wrapper: Java, Java and Kotlin API W-WasmThemis 🌐 Wrapper: WasmThemis, JavaScript API, npm packages
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ilammy @vixentael