Skip to content

Audit Logging: use the original url#81282

Merged
thomheymann merged 1 commit intomasterfrom
security/81133
Oct 28, 2020
Merged

Audit Logging: use the original url#81282
thomheymann merged 1 commit intomasterfrom
security/81133

Conversation

@thomheymann
Copy link
Copy Markdown
Contributor

Summary

Now that #80810 is merged, we should update our audit logging to use the KibanaRequest.rewrittenUrl property, instead of reconstructing this manually within the audit_events

Closes #81133

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@thomheymann thomheymann requested a review from a team as a code owner October 21, 2020 08:56
@thomheymann thomheymann added v7.11.0 v8.0.0 Team:Security Platform Security: Auth, Users, Roles, Spaces, Audit Logging, etc t// labels Oct 21, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-security (Team:Security)

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@thomheymann thomheymann merged commit 7e0b9ff into master Oct 28, 2020
@thomheymann thomheymann deleted the security/81133 branch October 28, 2020 11:37
gmmorris added a commit to gmmorris/kibana that referenced this pull request Oct 29, 2020
…kibana into alerts/convert-to-tm-intervals

* 'alerts/convert-to-tm-intervals' of github.com:gmmorris/kibana: (88 commits)
  fixed jest
  APM Experiments settings (elastic#81554)
  [Resolver] Enable resolver test plugin tests (elastic#81339)
  Add TS project references for inspector  (elastic#81792)
  Add uri decode to es_ui_shared and fix navigation issues with special characters (elastic#80835)
  [Fleet] Rename ingestManager translations fleet (elastic#81837)
  [Logs UI] Transmit and render array field values in log entries (elastic#81385)
  Audit Logging: use the original url (elastic#81282)
  [User experience] Fix JS error rate (elastic#81512)
  [UX] Add median/percentile info in titles (elastic#79824)
  Support export for SO with circular refs (elastic#81582)
  Get rid of  global types (elastic#81739)
  [APM] Fix precommit script (elastic#81594)
  skips overview tests (elastic#81877)
  [Security Solution][Case] Fix connector's labeling (elastic#81824)
  Added simple test, which only covers successful case when edit happened right after task was complete previous execution
  [Maps] Fix EMS test (elastic#81856)
  [Security Solutions][Detections] - Fix bug, last response not showing for disabled rules (elastic#81783)
  skip flaky suite (elastic#81853)
  Fixed type checks and unit tests
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:fix Team:Security Platform Security: Auth, Users, Roles, Spaces, Audit Logging, etc t// v7.11.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Audit Logging: use the original url provided by core

4 participants