Skip to content

Remove explicit logging of GraphServiceException #885

Description

CoreHttpProvider.processResponse catches GraphServiceExceptions and writes the exception explicitly with logger.error.

This is not needed because the exception is also thrown afterwards (and can be handled individually by the caller).

The explicit logging should be removed so that the GraphServiceException is no longer written to log in each case.

image

Activity

  1. baywet commented on Jun 7, 2023

    @baywet
    Member

    Thanks for the suggestion. Would you be willing to submit a pull request to address this?

  2. MaHa6543 commented on Jun 7, 2023

    @MaHa6543
    ContributorAuthor

    Thanks for the quick response. I submitted pull request #898

  3. added a commit that references this issue on Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions