Articulate 5.0.1 causes all node URLs except the homepage to return 404 not found

[SRUC]

Articulate 5.0.1 causes all node URLs except the homepage to return 404 not found.

Tested using Umbraco.TheStarterKit and OskiStarterKit in Umbraco v12.3.3 and v13.0.0 RC2

After removing Articulate 5.0.1 and rebuilding the solution, node URLs function as expected.

[Shazwazza]

What root nodes do you have then with all of these starter kits? Do you have multiple root nodes? Do you have domains defined for them? There can only be a single root node without a domain assigned in Umbraco.

[gavinfaux]

Have seen same issue on a previous working 5.0.0 site, two root nodes, first node has domain / culture assigned, second node nothing assigned (container for content blocks and settings). Uninstall of 5.0.1 and reverting to 5.0.0 allows rest of site to render. Have not looked into further as yet.

[khraibani]

Same issue happening with version 5.0.1 as @gavinfaux

[Shazwazza]

@khraibani exactly the same? 2 root nodes, one domain assigned, or something different?

[MartinAmsinck]

I ran in to something similar i think.
I'm on a 13.0.3, and i'm using a fork of Articulate, not an installed package.

I have multiple Sites, under one rootnode, and each of them has its own domain

But nothing but the first rootnode gets hit (the "Sites" node), everything else is a 404.
The "Sites" node has no domain.

When removing the Articulate code, everything works as expected.

It seems that when hitting this part of the code in ArticulateRouteValueTransformer.ShouldCheck(), umbracoRouteValues?.PublishedRequest is null

The comment Ensure it runs AFTER Umbraco made me bump the value from 100 to 1000.
Seems to do the trick for me.

Hope this helps

[nvisage-gf]

@MartinAmsinck Thanks for suggested fix, our site working fine now, PR submitted.

Unsure how to debug/list route order to verify if 1000 is optimal.

[khraibani]

@Shazwazza I was running

• Umbraco v12.3.6
• Articulate 5.0.1
• 1 root node
  I reverted back to version 5.0.0 and its working apart from the categories/tags I will raise that as another issue

[Shazwazza]

@khraibani are you running in Umbraco Cloud? I tweeted about this a while back https://twitter.com/Shazwazza/status/1720142890901684630 and had to ship 5.0.1 to make it work for the virtual routes.

[Shazwazza]

Though, maybe my workarounds (see 6d0b672, e3d34ba) was resolved by changing the MapDynamicControllerRoute to 1000, not sure

[khraibani]

@Shazwazza no its a local version of Umbraco (self hosted) should that make a difference?

[eqtr-ab]

Hey there, we encountered this issue today while adding v5.0.1 of Articulate to an Umbraco Cloud site running v13.0.3 of the CMS. Using v5.0.0 instead resolves the issue.

[BarryGibson]

I too had the same issue. I found that the pages were available if I added // after the base domain i.e. https://localhost:44372//Page-name.
This was fixed in the latest Dev version as well as using V5.0.0.

Notes to help debug if needed:
It only started happening after I added a custom SurfaceController.
I confirmed this by creating a clean Umbraco 13.1 build
The site worked correctly with Articulate V5.0.1 without a SurfaceController and with a SurfaceController but not Articulate but with both it returned "404 not found" - the 404 wasn't an Umbraco handled 404 it was a system 404.
For testing the SurfaceController was a pretty empty one as shown below.

using Microsoft.AspNetCore.Mvc;
using Umbraco.Cms.Core.Cache;
using Umbraco.Cms.Core.Logging;
using Umbraco.Cms.Core.Routing;
using Umbraco.Cms.Core.Services;
using Umbraco.Cms.Core.Web;
using Umbraco.Cms.Infrastructure.Persistence;
using Umbraco.Cms.Web.Website.Controllers;

namespace UmbracoProject1.Controllers
{
	public class SampleController : SurfaceController
	{
		public SampleController(IUmbracoContextAccessor umbracoContextAccessor, IUmbracoDatabaseFactory databaseFactory, ServiceContext services, AppCaches appCaches, IProfilingLogger profilingLogger, IPublishedUrlProvider publishedUrlProvider) : base(umbracoContextAccessor, databaseFactory, services, appCaches, profilingLogger, publishedUrlProvider)
		{
		}

		public IActionResult Index()
		{
			return Content("Hello world");
		}
	}
}

[JamesDodwell]

I've encountered this issue too and the // after the base url does mean the pages load e.g. https://localhost:44310//articles/latte-art/

[westinlennoxbrandner]

We are seeing this same issue with 5.0.1. Our solution was to downgrade to 5.0.0 which has fixed the issue.

[The-algar]

Had the same issue Umbraco CMS 13.2.2, Articulate latest version, One root node
once Articulate installed, 404 error for all pages
Question: How to Setup An Articulate Blog Package to An Existing Umbraco Layout
Is there's a process on how to do this?

[Shazwazza]

really sorry for the accidental delay in getting these shipped as part of releases, I though this was already done 🤦‍♂️🤦‍♂️ 5.0.2 and 5.0.3 are now shipped.

I'm not sure if those 'fix' this issue but hopefully will help some folks, I still need to replicate locally

[BarryGibson]

really sorry for the accidental delay in getting these shipped as part of releases, I though this was already done 🤦‍♂️🤦‍♂️ 5.0.2 and 5.0.3 are now shipped.

I'm not sure if those 'fix' this issue but hopefully will help some folks, I still need to replicate locally

Thanks this is now fixed when using 5.0.3