Skip to content

Support both, encoded and non encoded api-key formats on plugin configuration#203

Merged
alexcams merged 5 commits intologstash-plugins:mainfrom
alexcams:202-support-opaque-api_key
Sep 23, 2025
Merged

Support both, encoded and non encoded api-key formats on plugin configuration#203
alexcams merged 5 commits intologstash-plugins:mainfrom
alexcams:202-support-opaque-api_key

Conversation

@alexcams
Copy link
Contributor

Previously, API keys that were already in encoded format were incorrectly re-encoded, causing connection failures. This update ensures that only raw keys are encoded, supporting both opaque and id:api-key formats without issue.

Closes #202

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

@alexcams alexcams marked this pull request as draft September 17, 2025 15:06
@alexcams alexcams marked this pull request as ready for review September 22, 2025 11:18
@alexcams alexcams force-pushed the 202-support-opaque-api_key branch from 4f5d406 to 3b1ee4f Compare September 22, 2025 11:22
@alexcams alexcams force-pushed the 202-support-opaque-api_key branch 3 times, most recently from 4bc7fcd to ab01dd3 Compare September 23, 2025 08:43
@alexcams alexcams requested a review from andsel September 23, 2025 10:54
@alexcams alexcams force-pushed the 202-support-opaque-api_key branch from ab01dd3 to 85a853b Compare September 23, 2025 11:35
Copy link
Contributor

@andsel andsel left a comment

Choose a reason for hiding this comment

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

LGTM

@alexcams alexcams merged commit 824415f into logstash-plugins:main Sep 23, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants