Redirect Url Service Notifications in v13

ProNotion · April 2, 2025, 1:46pm

I need to create a custom webhook for notifications of when a new Redirect Url is added or removed. In order to implement this I need to create some custom notifications but for all of this to work as expected those notifications will need to fire when the register and delete operations occur in the RedirectUrlService. There is just one small problem - RedirectUrlService is internal and there are currently no notifications in this service that I can see.

Is my only option here to implement my own IRedirectUrlService? I was kind of hoping I could inherit the core one and do what I needed but I don’t think I can do that.

JasonElkin · April 3, 2025, 10:11am

I’m afraid you’ll need to roll your own IRedirectUrlService. Fortunately the RedirectUrlService is only a thin wrapper around the IRedirectUrlRepository so there’s not a lot to duplicate.

github.com/umbraco/Umbraco-CMS

src/Umbraco.Core/Services/RedirectUrlService.cs

contrib

using System.Threading.Tasks;
using Microsoft.Extensions.Logging;
using Umbraco.Cms.Core.Events;
using Umbraco.Cms.Core.Models;
using Umbraco.Cms.Core.Persistence.Repositories;
using Umbraco.Cms.Core.Scoping;

namespace Umbraco.Cms.Core.Services;

internal class RedirectUrlService : RepositoryService, IRedirectUrlService
{
    private readonly IRedirectUrlRepository _redirectUrlRepository;

    public RedirectUrlService(
        ICoreScopeProvider provider,
        ILoggerFactory loggerFactory,
        IEventMessagesFactory eventMessagesFactory,
        IRedirectUrlRepository redirectUrlRepository)
        : base(provider, loggerFactory, eventMessagesFactory) =>
        _redirectUrlRepository = redirectUrlRepository;

This file has been truncated. show original

I suspect adding notifications and/or webhook support to the RedirectUrlService would be a welcome PR to Umbraco too - this is becoming more important with headless setups and for offloading redirects to edge servers.

ProNotion · April 3, 2025, 12:16pm

@JasonElkin Thank you for your reply. Happy to submit a PR for this if there is a chance it can make it into v13.

I’m not too concerned about the amount of work to duplicate in a custom RedirectUrlService it is more that the code will almost certainly deviate from the core down the line and so a direct inheritance would have been preferred.