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

Log more response errors #891

Merged
7 commits merged into from
Sep 19, 2023
Merged

Log more response errors #891

7 commits merged into from
Sep 19, 2023

Commits on Sep 18, 2023

  1. fix: log more response errors 

    Previously we only logged (not sure but current assumption) BAD_REQUEST returned from saas
    @Zelldon
    Zelldon committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    7aee8be View commit details
    Browse the repository at this point in the history

  2. refactor: rename class and log error

    @Zelldon
    Zelldon committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    0895e7a View commit details
    Browse the repository at this point in the history

  3. test: verify that exception is thrown

    @Zelldon
    Zelldon committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    9d31c92 View commit details
    Browse the repository at this point in the history

  4. test: rename test method

    @Zelldon
    Zelldon committed Sep 18, 2023
    Configuration menu
    Copy the full SHA
    beae2dd View commit details
    Browse the repository at this point in the history

Commits on Sep 19, 2023

  1. refactor: rename class

    @Zelldon
    Zelldon committed Sep 19, 2023
    Configuration menu
    Copy the full SHA
    33648ef View commit details
    Browse the repository at this point in the history

  2. refactor: make object mapper static

    @Zelldon
    Zelldon committed Sep 19, 2023
    Configuration menu
    Copy the full SHA
    05f7bdf View commit details
    Browse the repository at this point in the history

  3. test: remove test 

    This test was more to verify how the behavior is currently when an exception or error response is sent, but doesn't bring much value since it is already tested in the dependencies
    @Zelldon
    Zelldon committed Sep 19, 2023
    Configuration menu
    Copy the full SHA
    0e3352a View commit details
    Browse the repository at this point in the history