Add robustness to monitoring of workflow URLs.

[philliphoff]

Description

Issue reference

Please reference the issue this PR will close: #7087

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

• Code compiles correctly
• Created/updated tests
• Unit tests passing
• End-to-end tests passing
• Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
• Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
• Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

[ItalyPaleAle]

LGTM but can you rebase this on release-1.12? Since for master, the issue will be fixed by #6723

[yaron2]

I think we can do with this fix (or the other PR) for master only, not entirely sure this needs to go into a patch release as Workflows is beta and this isn't a showstopper to using the APIs

[ItalyPaleAle]

I think we can do with this fix (or the other PR) for master only,

In master, please don't merge this as it will just cause more conflicts on #6723 which then will need to be fixed

Workflows is beta and this isn't a showstopper to using the APIs

While that's true, it's causing daprd to panic so it could even be considered a DoS

[philliphoff]

I'll rebase it to release-1.12.

[ItalyPaleAle]

LGTM but it's up to the maintainers to accept this

[daixiang0]

Approve, wait for DCO fix.

[philliphoff]

@daixiang0 I'm not sure what GH is complaining about; it's a single commit that contains the sign off (for me), as best I can tell.

[ItalyPaleAle]

@philliphoff i think this PR is in a broken state after the rebase. Could you try force-pushing your commit again?

[mukundansundar]

@philliphoff can you fix the lint error that is seen here?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c1e5a0a) 64.76% compared to head (5c37764) 64.73%.

Additional details and impacted files

@@               Coverage Diff                @@
##           release-1.12    #7094      +/-   ##
================================================
- Coverage         64.76%   64.73%   -0.03%     
================================================
  Files               231      231              
  Lines             20940    20942       +2     
================================================
- Hits              13561    13557       -4     
- Misses             6250     6252       +2     
- Partials           1129     1133       +4     

Files	Coverage Δ
pkg/diagnostics/http_monitoring.go	79.37% <100.00%> (+0.26%)	⬆️

... and 7 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.