Skip to content
This repository has been archived by the owner on Jan 27, 2020. It is now read-only.

Increase max dynhost password length from 12 to 31 #987

Merged
merged 1 commit into from Apr 8, 2019

Conversation

trnsnt
Copy link
Contributor

@trnsnt trnsnt commented Apr 5, 2019

Increase max dynhost password length from 12 to 31

Description of the Change

Increase max dynhost password length from 12 to 31 to be coherent with API.

Benefits

Be coherent with API.

Possible Drawbacks

None

Applicable Issues

None

@antleblanc
Copy link
Contributor

antleblanc commented Apr 8, 2019

Thanks @trnsnt for your contribution.

Do you know if theses information can be retrieve directly from the API instead?

It will help if theses data changes over the time.

Could you also please target develop branch?

Thanks!

ref: MANAGER-2459

@antleblanc antleblanc self-assigned this Apr 8, 2019
@trnsnt trnsnt changed the base branch from master to develop April 8, 2019 07:35
@trnsnt
Copy link
Contributor Author

trnsnt commented Apr 8, 2019

Hello @antleblanc, for now there is no way to get the password min / max length from the API. But it's something relatively stable over the time.
Moved to target develop !

Thx

@antleblanc antleblanc merged commit d197834 into ovh-ux:develop Apr 8, 2019
@antleblanc antleblanc mentioned this pull request Apr 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

None yet

2 participants