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

Wrong Documentation for requires_username #9278

Open
etwillbefine opened this issue Jul 28, 2020 · 0 comments
Open

Wrong Documentation for requires_username #9278

etwillbefine opened this issue Jul 28, 2020 · 0 comments

Comments

@etwillbefine
Copy link

etwillbefine commented Jul 28, 2020

Description

The Documentation says requires_username needs to be a property of validation. By following the Documentation Auth0 returns a Bad Request saying "Payload validation error: 'Additional properties not allowed: requires_username' on property options.validation (Options for validation).".

I actually like the idea of having requires_username within the validation field as it is currently documented.
So instead of opening a PR with Documentation Changes I first wanted to check in if there might be a version mismatch or if you are planning to support the documented data structure.

Reproduction

HTTP POST /api/v2/connections
{
	"name": "test2",
	"strategy": "auth0",
	"options": {
		"validation": {
		       "requires_username": true,
			"username": {
				"min": 1,
				"max": 5
			}
		}
	}
}

The current API Behaviour is to accept requires_username inside the options object.

Environment

Please provide the following:

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

No branches or pull requests

1 participant