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

Update CosmosHttpClientCore.cs #3044

Closed
wants to merge 6 commits into from
Closed

Conversation

anstein-msft
Copy link

Description

Adding context information to the exception thrown in the HTTP request made to add the User agent, when an invalid character is provided in the ApplicationName string when constructing a CosmosClient.

This will help developers figure out what issue they are dealing with when they provide an application name that is not valid.

Type of change

[] New feature (non-breaking change which adds functionality)

Closing issues

To automatically close an issue: closes #3040

@ghost
Copy link

ghost commented Feb 22, 2022

CLA assistant check
All CLA requirements met.

@anstein-msft
Copy link
Author

@kirankumarkolli @j82w @ealsur @FabianMeiswinkel @kirillg @sboshra

I reviewed my changes with another developer, made some updates, merged in all the recent changes to Cosmos Core. Please let me know if this is something that could get looked at to merge in.

@ealsur
Copy link
Member

ealsur commented Sep 14, 2022

Closing as the original issue is addressed by #3455

@ealsur ealsur closed this Sep 14, 2022
Sign up for free to 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.

Exception thrown when application name contains certain characters
2 participants