fix(influxdb): check authentication #11031
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
targeting
release-51
Checks authentication on bridge start and get status. Also, handle authentication error when sending message.
Fixes https://emqx.atlassian.net/browse/EMQX-10213
Summary
🤖 Generated by Copilot at 410faba
This pull request enhances the security and error handling of the emqx_bridge_influxdb connector by adding authentication checks and 401 error handling. It also adds test cases to verify the functionality and the expected logs and events. It modifies the
emqx_bridge_influxdb_connector.erl
andemqx_bridge_influxdb_SUITE.erl
files.PR Checklist
Please convert it to a draft if any of the following conditions are not met. Reviewers may skip over until all the items are checked:
changes/{ce,ee}/(feat|perf|fix)-<PR-id>.en.md
filesIf there should be document changes, a PR to emqx-docs.git is sent, or a jira ticket is created to follow upSchema changes are backward compatibleChecklist for CI (.github/workflows) changes
If changed package build workflow, pass this action (manual trigger)Change log has been added tochanges/
dir for user-facing artifacts update