grafana/synthetic-monitoring-app

Change authentication options for HTTP checks to a radio button pattern

ckbedwell opened this issue · 1 comments

What would you like to be added:
During HTTP check creation in the authentication section there are checkboxes to enable / disable bearer authorization and basic authorization -- the implication being that you can send both in one request if you tick both boxes. This is not possible as they share the same header.

It should be converted to a radio button pattern with three options 'None', 'Bearer' and 'Basic'.

Why is this needed:
To improve the UX for creating HTTP checks.

Additional context
If a user fills in one of the inputs and then changes their option, the UI state should remain (e.g. if they filled in the input for username and password and then clicked on bearer auth, then back to basic, the username and password they typed should remain) but it should not be sent in the payload if they have selected none or a different option.

Spoken with @majavojnoska. Please check with her what the design of this should be (currently researching existing patterns).