Skip to content

TypeError: NetworkError when attempting to fetch resource. #95

Closed

Description

@TrianguloY

When trying to fix the CORS error a few days ago, you made a change in the fetching code so now pages are fetched using fetch instead of fetchProxy.
adaa08a?branch=adaa08ac90758703e6294003b5a24a0e74286e17&diff=unified#diff-0f292e46fc1906eacb3fdbdaced082678d3b8d83fb44ab79b927d2abd836c358L107

Now, when using a page which has cors disabled, like dropbox, the service doesn't work.
http://htmlpreview.github.io/?https://www.dropbox.com/s/mgw65rktedx4pjl/Hello%20world.html?raw=1

Was that change a forgotten invalid fix? I recommend using fetchProxy again, so that dropbox links (and others) works again.

Activity

  1. niutech commented on Oct 13, 2020

    @niutech
    Collaborator

    This change was meant to lower the usage of CORS proxies and improve loading time, so that HTML documents from https://raw.githubusercontent.com could be fetched directly. I didn't realize that HTMLPreview is used outside of GitHub/Bitbucket, like with Dropbox. I will make an exception for such cases.

  2. TrianguloY commented on Oct 13, 2020

    @TrianguloY
    ContributorAuthor

    Ah, I understand.

    Yes, a friend of mine uses the page with dropbox (https://jdelcastilloshideout.blogspot.com/2020/09/paravicini-y-la-roca-rodante.html) and asked me why the page stopped working, so I investigated and found the issue.

    Thanks for the quick reply 😊. An exception is enough for my friend.

  3. added a commit that references this issue on Oct 16, 2020
    1e4aa6d
  4. TrianguloY commented on Oct 19, 2020

    @TrianguloY
    ContributorAuthor

    Sorry for the reply, but perhaps you have disabled notifications of pull requests.
    I investigated the issue and found a solution that also should drastically reduce the number of petitions to the cors proxies. More info in the pull request: #96

    You can check the result here:
    https://trianguloy.github.io/htmlpreview.github.com/?https://www.dropbox.com/s/mgw65rktedx4pjl/Hello%32world.html?raw=1
    https://trianguloy.github.io/htmlpreview.github.com/?https://github.com/twbs/bootstrap/gh-pages/2.3.2/index.html
    https://trianguloy.github.io/htmlpreview.github.com/?https://github.com/documentcloud/backbone/blob/master/examples/todos/index.html

  5. peastman commented on Oct 28, 2020

    @peastman

    I also started encountering this error, but my file is on Github. Here's the file:

    https://github.com/openmm/pdbfixer/blob/master/Manual.html

    The following link produces the error message given above:

    https://htmlpreview.github.io/?https://github.com/openmm/pdbfixer/blob/master/Manual.html

  6. peastman commented on Nov 9, 2020

    @peastman

    Is there anything I can do to work around this problem?

  7. TrianguloY commented on Nov 9, 2020

    @TrianguloY
    ContributorAuthor

    @peastman The link you posted above (https://htmlpreview.github.io/?https://github.com/openmm/pdbfixer/blob/master/Manual.html) works fine for me. I checked the mentioned issue from your repo and the link is different. Simply update the link to this one and it will work again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions