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
Clean up webserver endpoints adding to audit log #37580
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Looks like there might be a couple test failures from this change. |
Specifically |
Fixed. Removed two unneeded tests and added a test for robots.txt |
@jedcunningham For some reason |
714975b
to
2664251
Compare
* Clean up webserver endpoints adding to audit log * Remove all view-only action logging and change excluded events list * Restore robots logging * Remove two extra loggings+tests, add test for robots.txt * Fix rendered template test
* Clean up webserver endpoints adding to audit log * Remove all view-only action logging and change excluded events list * Restore robots logging * Remove two extra loggings+tests, add test for robots.txt * Fix rendered template test
Only log webserver events that are CRUD actions. View-only endpoints shouldn't be logged. Therefore we don't need any default excluded events.
Removed a few other unused decorators, mostly on on redirect functions.
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rst
or{issue_number}.significant.rst
, in newsfragments.