Checkmate icon indicating copy to clipboard operation
Checkmate copied to clipboard

Remove input validation for uptime monitoring

Open gorkem-bwl opened this issue 3 months ago • 5 comments

Some pages may need basic authentication (eg. https://<user>:<password>@<url>) and we need to allow this to pass the authentication.

gorkem-bwl avatar Sep 19 '25 03:09 gorkem-bwl

hey @gorkem-bwl, I'm interested to work on this issue. please will you give me more clarity about which routes need basic authentication except above mentioned URL. and will you assign this issue to me ?

Darshan98Solanki avatar Oct 28 '25 05:10 Darshan98Solanki

hey @gorkem-bwl, I'm interested to work on this issue. please will you give me more clarity about which routes need basic authentication except above mentioned URL. and will you assign this issue to me ?

TBH I have zero idea when and why I wrote this issue lol.

@ajhollid can you please give more context about this issue, and whether it's still valid/related?

gorkem-bwl avatar Oct 28 '25 13:10 gorkem-bwl

hey @gorkem-bwl, I'm interested to work on this issue. please will you give me more clarity about which routes need basic authentication except above mentioned URL. and will you assign this issue to me ?

TBH I have zero idea when and why I wrote this issue lol.

@ajhollid can you please give more context about this issue, and whether it's still valid/related?

not an issue @ajhollid please give me some context about it.

Darshan98Solanki avatar Oct 28 '25 16:10 Darshan98Solanki

The issue is that users would like to remove input validation for the URL field when creating monitors so that basic auth can be added to monitor URLs.

ajhollid avatar Oct 28 '25 16:10 ajhollid

This would actually be really usefull. 👍 The URL input for an uptime monitor should not be validated to allow to enter URLs that e.g. contain basic authentification credentials to be able to monitor pages that require basic authentification.

This example should actually be valid:

Image

der-On avatar Nov 05 '25 20:11 der-On