Progress endpoint won't respect PrefixPath defined in dashboard UI

[eCorke]

It seems that progress circular loading that's shown in overview of running tasks (for each row) doesn't respect PrefixPath defined in dashboard registration. The repeated requests towards hangfire/console/progress is never prefixed.

As in my case, the dashboard is exposed on sub address (https://localhost:6001/application/hangfire...) with PrefixPath defined as application, the progress requests point to https://localhost:6001/hangfire/console/progress

[khellang]

The code just uses default Hangfire APIs to register dashboard routes:

Hangfire.Console/src/Hangfire.Console/GlobalConfigurationExtensions.cs

Lines 44 to 46 in a1fea13

	// register dispatchers to serve console data
	DashboardRoutes.Routes.Add("/console/progress", new JobProgressDispatcher(options));
	DashboardRoutes.Routes.Add("/console/([0-9a-f]{11}.+)", new ConsoleDispatcher(options));

I think this might be a bug in Hangfire itself?

[AnderssonPeter]

@khellang I saw that you created a fork of this repository, I have a nuget package that extends this one and was thinking about creating 2 additional nugets that target your fork.
Is it ok if I named them IdentityStream.Hangfire.Console.Extensions and IdentityStream.Hangfire.Console.Extensions.Serilog? (If it's your company name I fully understand if your not OK with it!!)

https://github.com/AnderssonPeter/Hangfire.Console.Extensions
https://www.nuget.org/packages/Hangfire.Console.Extensions
https://www.nuget.org/packages/Hangfire.Console.Extensions.Serilog

[khellang]

I'm not totally opposed to it, but the IdentityStream.* prefix is reserved for my company's account on NuGet.org, so I'm not sure how doable it is. Any other suggestions? 😆

[AnderssonPeter]

If that is the case how about Hangfire.Console.Extensions.IdentityStream and Hangfire.Console.Extensions.Serilog.IdentityStream?

[khellang]

Yeah, that's fine 👍🏻

[AnderssonPeter]

@khellang seems like anything with the name IdentityStream in it is reserved.
How about if you create a API Key that only allows me to push IdentityStream.Hangfire.Console.*? (You can specify a pattern when you create the key)

[khellang]

Hmm. That's weird. I wonder what part of "prefix" Microsoft doesn't get? 😆 The prefix is explicitly IdentityStream.*:

I'm not too keen on having packages we don't maintain sitting under the IdentityStream NuGet organization either 🤔

[khellang]

What about IdS.Hangfire.Console.Extensions or Hangfire.Console.Extensions.IdS? 😅

[gilmar-sales]

Hi, I think my PR on IdentityStream Repo fixes this: IdentityStream#5

It's simple, just use the UrlHelper and let things go.

[gilmar-sales]

I'll made the same PR here