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.

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

RequestFailedException: decouple Message and what(). #3201

Merged
merged 1 commit into from
Dec 18, 2021
Merged

RequestFailedException: decouple Message and what(). #3201

merged 1 commit into from
Dec 18, 2021

Conversation

antkmsft
Copy link
Member

@antkmsft antkmsft commented Dec 17, 2021

Closes #2542.

  1. Rename whatArg to what per Jeffrey's feedback: Rename exceptions message parameter to whatArg #2782 (comment)
  2. The only user of RequestFailedException(what, response) is the unit test.
  3. Delegate construction to a constructor that makes the best effort to parse the response.
  4. what is what we set programmatically, Message is what gets returned from the service.

@antkmsft antkmsft self-assigned this Dec 17, 2021
@antkmsft antkmsft added Azure.Core MQ This issue is part of a "milestone of quality" initiative. labels Dec 17, 2021
@antkmsft antkmsft added this to the [2022] January milestone Dec 17, 2021
@antkmsft
Copy link
Member Author

/azp run cpp - core

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@antkmsft
Copy link
Member Author

/azp run cpp - storage

@antkmsft
Copy link
Member Author

/azp run cpp - keyvault

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@antkmsft
Copy link
Member Author

/azp run cpp - identity

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@antkmsft antkmsft merged commit b4fe751 into Azure:main Dec 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure.Core MQ This issue is part of a "milestone of quality" initiative.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename the message param to what_arg on RFE following the std::runtime_error base class ctor
2 participants