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.

Sign up for GitHub

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

Jump to bottom

Fix Audit Message Logging Interceptor Race Condition #938

Merged
merged 18 commits into from
Sep 7, 2020
Merged

Fix Audit Message Logging Interceptor Race Condition #938

merged 18 commits into from
Sep 7, 2020

Commits on Sep 4, 2020

  1. Fix race condition in GrpcMessageInterceptor to revert a empty messag… 

    …e if message cannot be recorded.
    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    90b922a View commit details
    Browse the repository at this point in the history

  2. Fix GrpcMessageInterceptor race condition by allow audit log message … 

    …to be called from multiple async calls
    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    88f2210 View commit details
    Browse the repository at this point in the history

  3. Revert option change

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    e8861a6 View commit details
    Browse the repository at this point in the history

  4. Add CoreLoggingIT integration test to test message audit logging.

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    2825407 View commit details
    Browse the repository at this point in the history

  5. Fix GrpcMessageInterceptor race condition by moving allowing request … 

    …to be unset.
    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    db3e5dc View commit details
    Browse the repository at this point in the history

  6. Fix lint

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    42fe1dc View commit details
    Browse the repository at this point in the history

  7. Increase wait for logs in CoreLoggingIT

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    469b647 View commit details
    Browse the repository at this point in the history

  8. Fix to compare the correct lob JsonObject with the right response.

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    8de5e56 View commit details
    Browse the repository at this point in the history

  9. Debug response

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    b3a1acd View commit details
    Browse the repository at this point in the history

  10. Reduce load size to make test less flaky

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    e6bb39b View commit details
    Browse the repository at this point in the history

  11. Update test to only check request and response for one call.

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    d6ded00 View commit details
    Browse the repository at this point in the history

  12. Fix compile failure due to uncaught exception.

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    74b07bc View commit details
    Browse the repository at this point in the history

  13. Add method name filter to prevent logs from tests from interfering wi… 

    …th each other.
    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    297da1f View commit details
    Browse the repository at this point in the history

  14. Add intergration test to check that message logs are produced correct… 

    …ly under load.
    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    03b7eb1 View commit details
    Browse the repository at this point in the history

  15. Fix imports and log4j2 not able to find config file

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    e1b173f View commit details
    Browse the repository at this point in the history

  16. Fix issue with CoreLoggingIT TestLogAppender being null due to class … 

    …not found by log4j2
    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    81135c2 View commit details
    Browse the repository at this point in the history

  17. Remove unused getters in MessageAuditLogEntry.

    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    794e8c4 View commit details
    Browse the repository at this point in the history

  18. Update CoreLoggingIT test to check that expected contents of logs pro… 

    …duced under load.
    @mrzzy
    mrzzy committed Sep 4, 2020
    Configuration menu
    Copy the full SHA
    fc75f83 View commit details
    Browse the repository at this point in the history