Gatsby: Navigating from 404 page errors on production deployment

[gtfargo]

When a user attempts to navigate _from_ the 404 page on a production build, the console shows this error and the page never redirects:

Uncaught TypeError: Cannot read property 'push' of undefined
    at eventHandler (production-app.js:51)
    at mitt.js:1
    at Array.map (<anonymous>)
    at Object.emit (mitt.js:1)
    at done (loader.js:284)
    at loader.js:295
    at loader.js:95
    at loader.js:78
    at index.js?d7c6:11
    at handler (patch.js:18)

or, if content for the desired destination is already loaded:

Cannot read property 'push' of undefined
    at navigateTo (production-app.js:66)
    at t.navigateTo (index.js:154)
    at gatsby-browser.js:12
    at catch-links.js:51

404.js

import React from 'react';
import Link from 'gatsby-link';

const NotFoundPage = () =>
  <div>
    <h1>NOT FOUND</h1>
    <Link to={`/`}>Click here to go home</Link>
  </div>;

export default NotFoundPage;

dependency versions:

"gatsby": "^1.8.11",
"gatsby-link": "^1.6.7",
"gatsby-plugin-catch-links": "^1.0.4",

[kbariotis]

I have the same error using a gatsby-link at my 404 component.

[gtfargo]

Attempted to reproduce with a the boilerplate example... Seems my personal deployment is causing the issue. Still digging into this for more info.

[gtfargo]

Seems the most recent update has fixed this, @kbariotis try updating.

[kbariotis]

@gtfargo hmm not really, still the same issue.

[mikhail-softgrad]

I have same issue
@KyleAMathews

[KyleAMathews]

One of you want to take a crack at fixing this?

[kbariotis]

@KyleAMathews Sure I will. Could you please guide me to the right direction in the codebase, since I haven't touched it before?

[gtfargo]

In the if statement of this eventHandler in production-app.js the event value is _only_ the to value of the gatsby-link. In my case, "/"

function eventHandler(e) {
      if (e.page.path === loader.getPage(pathname).path) {
        emitter.off(`onPostLoadPageResources`, eventHandler)
        clearTimeout(timeoutId)
        window.___history.push(pathname)
      }
    }

[KyleAMathews]

The error is happening in https://github.com/gatsbyjs/gatsby/blob/master/packages/gatsby/cache-dir/production-app.js

Follow the contributing instructions so you can develop against the checked out version of Gatsby and start adding console.logs to understand why there's the failure.

[attfarhan]

@KyleAMathews @kbariotis @gtfargo I also just ran into this issue. Anything we can do to get the PR updated and merged soon?

[kbariotis]

Im sorry @attfarhan didn't find the time to work on it :/ I need it too so I will try to check it again soon.

[revolunet]

same here; some more info; quite critical because once the user lands on a 404, he can never navigate again

[johnryan]

fwiw this doesn't just affect links, it seems like none of the JS is being loaded in, (i.e. code called in componentDidMount is executed). @KyleAMathews is this a separate issue or is this resolved in your PR?

[KyleAMathews]

This is still an issue.

[efallancy]

@KyleAMathews I can get this done if you don't mind 😃 This particularly happened when there's no history being pushed to window.___history

[KyleAMathews]

That'd be great!

On Sat, Oct 28, 2017, 6:36 PM Emmanuel Fallancy [e-mail]
wrote:

@KyleAMathews https://github.com/kyleamathews I can get this done if
you don't mind 😃 This particularly happened when there's no history
being pushed to window.___history

—
You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
https://github.com/gatsbyjs/gatsby/issues/1838#issuecomment-340231314,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEVhx74TnYQtMCmOpE_ykrpSH4DJT3vks5sw9apgaJpZM4O5aAe
.

[revolunet]

in the meantime, any idea how to disable client-side on pages/404.js only ?

[efallancy]

@revolunet do you mean as not using the pages/404.js page in production?

[revolunet]

no i mean, juste disable the client-side javascript on that page so links are clickable :)

btw, i see PR https://github.com/gatsbyjs/gatsby/pull/2667 is getting some love, so it may be better to focus on that real fix :)

[efallancy]

ah, right 😅 i see what you mean.

[mickeyreiss-visor]

Has anyone made meaningful progress on the fix? This is a user-impactful bug.

@KyleAMathews Do you have a sense of where the issue lies? I'd be willing to take a try at the fix.

[KyleAMathews]

@mikhail-softgrad this is fixed actually :-) @emmafallancy did several PRs a month or so ago so closing this now.

[mickeyreiss-visor]

Thanks, @KyleAMathews!

For others in a similar situation, it looks like @emmafallancy's fix landed via https://github.com/gatsbyjs/gatsby/pull/2667 and was released in [email protected].

[janosh]

It appears as though this issue was not yet fixed in v2. Can someone confirm this?