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

Update elastic-agent-libs to v0.18.0 #42202

Merged
merged 2 commits into from
Jan 3, 2025

Conversation

michel-laterman
Copy link
Contributor

@michel-laterman michel-laterman commented Jan 2, 2025

Proposed commit message

Update elastic-agent-libs to v.0.18.0

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Disruptive User Impact

elastic-agent-libs v0.18.0 updates the min accepted TLS version to 1.1, and sets the default min to 1.2.

Related Issues

@michel-laterman michel-laterman added release-note:skip The PR should be ignored when processing the changelog dependency backport-8.x Automated backport to the 8.x branch with mergify labels Jan 2, 2025
@michel-laterman michel-laterman requested a review from a team as a code owner January 2, 2025 17:41
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 2, 2025
@botelastic
Copy link

botelastic bot commented Jan 2, 2025

This pull request doesn't have a Team:<team> label.

@jlind23
Copy link
Collaborator

jlind23 commented Jan 3, 2025

@mauri870 agentbeat tests are failing with the error below, do you know what can cause this?

../filebeat/fbreceiver/receiver_test.go:133:26: NewFactory().CreateLogsReceiver undefined (type receiver.Factory has no field or method CreateLogsReceiver)

@mauri870
Copy link
Member

mauri870 commented Jan 3, 2025

@mauri870 agentbeat tests are failing with the error below, do you know what can cause this?

This should be fixed by #42199, looks like it is already merged.

@jlind23 jlind23 merged commit ef29005 into elastic:main Jan 3, 2025
141 checks passed
mergify bot pushed a commit that referenced this pull request Jan 3, 2025
Co-authored-by: Julien Lind <[email protected]>
(cherry picked from commit ef29005)

# Conflicts:
#	go.mod
#	go.sum
@andrewkroh
Copy link
Member

@michel-laterman Have you made documentation changes related to this change? I assume the SSL docs need updates for both the supported_protocols and cipher_suites (like purging ciphers that only work for TLS 1.0 if there are any).

@michel-laterman michel-laterman deleted the update-elastic-agent-libs branch January 8, 2025 17:52
@michel-laterman
Copy link
Contributor Author

Thanks @andrewkroh, I've made #42267 to update the beats docs and elastic/ingest-docs#1609 for agent/fleet-server.

I'm not aware of any cipher_suites that are disabled with the newer revisions of TLS.

jlind23 added a commit that referenced this pull request Jan 9, 2025
* Update elastic-agent-libs to v0.18.0 (#42202)

Co-authored-by: Julien Lind <[email protected]>
(cherry picked from commit ef29005)

# Conflicts:
#	go.mod
#	go.sum

* Fix backport

---------

Co-authored-by: Michel Laterman <[email protected]>
Co-authored-by: michel-laterman <[email protected]>
Co-authored-by: Julien Lind <[email protected]>
@andrewkroh
Copy link
Member

andrewkroh commented Jan 9, 2025

I'm not aware of any cipher_suites that are disabled with the newer revisions of TLS.

That's probably true. It's not until TLS 1.3 that the weak cipher suites were dropped. With TLS 1.2 still being supported, there's likely none to remove at this time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8.x Automated backport to the 8.x branch with mergify dependency needs_team Indicates that the issue/PR needs a Team:* label release-note:skip The PR should be ignored when processing the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove TLS 1.x from default list of supported protocols
4 participants