Skip to content

Conversation

@calvinsID
Copy link
Contributor

@calvinsID calvinsID commented Jan 21, 2021

Description
Fixes #16622
Run az webapp up/create with a name that has a period (".")
image

Made this fix on CLI instead of the checkNameAvailability API because "." is valid for a full site hostname (URL's allow periods), just not valid for an appservice name.
I think portal also does some frontend validations like this when creating a webapp (though I haven't confirmed this)

Testing Guide
Expected msg:
image


This checklist is used to make sure that common guidelines for a pull request are followed.

@calvinsID calvinsID added this to the S182 milestone Jan 21, 2021
Copy link
Contributor

@panchagnula panchagnula left a comment

Choose a reason for hiding this comment

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

LGTM

@qwordy
Copy link
Member

qwordy commented Jan 22, 2021

@calvinsID You updated your profile picture. 😎

@qwordy qwordy merged commit 97107f2 into Azure:dev Jan 22, 2021
@calvinsID
Copy link
Contributor Author

haha yeah @qwordy it was about time i updated

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.

WebApp: az webapp up/create gives confusing message when using name with a period

3 participants