Skip to content

Log message of GraphServiceException only once - #514

Merged
Vincent Biret (baywet) merged 2 commits into
microsoftgraph:devfrom
qavid:bugfix/gh-480
May 25, 2022
Merged

Log message of GraphServiceException only once #514
Vincent Biret (baywet) merged 2 commits into
microsoftgraph:devfrom
qavid:bugfix/gh-480

Conversation

@qavid

Copy link
Copy Markdown
Contributor

Fixesgh-480

@baywetVincent Biret (baywet) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks a lot for the contribution!

@ghost

Deleted user (ghost) commented May 25, 2022

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@baywet

Copy link
Copy Markdown
Member

qavid would you mind singing the CLA please? (see the message from the bot in the thread)

@qavid

Copy link
Copy Markdown
ContributorAuthor

Vincent Biret (@baywet) CLA signed

@baywet

Copy link
Copy Markdown
Member

Thanks for signing the CLA. The PR is currently failing because of an outdated dependency. I've put #515 to address that. Once it gets merged, we'll be able to rebase this one and we should be able to merge it.

@baywet
Vincent Biret (baywet) merged commit 355822f into microsoftgraph:devMay 25, 2022
@baywetVincent Biret (baywet) added this to the 2.0.13 milestone May 25, 2022
@qavid
qavid deleted the bugfix/gh-480 branch May 25, 2022 20:02
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DefaultLogger's implementation for http 404 make no sense

2 participants

@qavid@baywet