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

Strip Azure EventHub connection string in debug logs #25066

Merged
merged 2 commits into from
Apr 14, 2021

Conversation

exekias
Copy link
Contributor

@exekias exekias commented Apr 14, 2021

No description provided.

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Apr 14, 2021
@exekias exekias added the Team:Integrations Label for the Integrations team label Apr 14, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Apr 14, 2021
Copy link
Member

@jsoriano jsoriano left a comment

Choose a reason for hiding this comment

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

Looks good, only wondering about the use of (redacted) instead of the endpoint if it doesn't have parameters.

@elasticmachine
Copy link
Collaborator

elasticmachine commented Apr 14, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #25066 updated

  • Start Time: 2021-04-14T08:56:25.043+0000

  • Duration: 53 min 3 sec

  • Commit: b3635a7

Test stats 🧪

Test Results
Failed 0
Passed 6919
Skipped 1185
Total 8104

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 6919
Skipped 1185
Total 8104

@exekias exekias merged commit 96fa570 into elastic:master Apr 14, 2021
exekias pushed a commit to exekias/beats that referenced this pull request Apr 14, 2021
* Strip Azure EventHub connection string in debug logs

(cherry picked from commit 96fa570)
exekias added a commit to exekias/beats that referenced this pull request Apr 14, 2021
Fixes changelog reference for PR elastic#25066
@exekias exekias mentioned this pull request Apr 14, 2021
exekias pushed a commit that referenced this pull request Apr 14, 2021
Fixes changelog reference for PR #25066
exekias pushed a commit that referenced this pull request Apr 15, 2021
…debug logs (#25077)

* Strip Azure EventHub connection string in debug logs (#25066)
v1v added a commit to v1v/beats that referenced this pull request Apr 15, 2021
* upstream/master:
  packer cache support for the 7.x and 7.latestMinor branches (elastic#25091)
  Remove EventFetcher and EventsFetcher interface (elastic#25093)
  Update go-structform to 0.0.8 (elastic#25051)
  Update copy_fields.asciidoc (elastic#25053)
  [elastic-agent] ensure container is backwards compatible (elastic#25092)
  Add --fleet-server-service-token. Rename --fleet-server to --fleet-server-es. (elastic#25083)
  Add cgroup.cpuacct percentages (elastic#25057)
  Add tests for truncated and symlinked files in filestream input (elastic#24425)
  Fix panic when Hearbeat monitor initialization fails twice (elastic#25073)
  [Filebeat][httpjson] Change append transform to initiate new fields as a slice (elastic#25074)
  Osquerybeat: Result values type translation (elastic#25012)
  Update Osquerybeat spec to get it downloading from the correct artifactory path (elastic#25076)
  Fix changelog (elastic#25079)
  Strip Azure EventHub connection string in debug logs (elastic#25066)
  Change googlecloud to gcp in field names (elastic#25038)
  Bump stack version to 7.12.0 for testing (elastic#24957)
  packer-cache: cache the existing docker images on ARM and some more (elastic#25068)
  Disable logstash TestFetch flaky test (elastic#25044)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Integrations Label for the Integrations team v7.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants