Skip to content

Improve generic error handling #82

Open

Description

@kjetilk

There have been issues with the "Log In" button in the popup being unresponsive. I still have this problem, nodeSolidServer/node-solid-server#884 and others had this problem due to an expired cert.

There should be some kind of feedback to the user in this situation, at least an error that they can go to the forum about. Perhaps there should even be a forum link in the error message if it is very generic?

Activity

  1. RubenVerborgh commented on Oct 26, 2018

    @RubenVerborgh
    Contributor

    I think the problem is that the feedback never reaches the client code. Fetchers in the browser are not allowed to see many details about request failures, as these can be exploited.

    But I can probably set a timeout that results in an error message.

    Can someone post exact steps to reproduce?

  2. kjetilk commented on Oct 26, 2018

    @kjetilk
    MemberAuthor

    Hmmm, but in my case, it happens without any network interaction, and it only happens on my laptop, so it is clearly highly environment dependent... Awfully hard to reproduce... I'll tinker with some plugins and see if that matters, at least.

    In JD's case, they had an invalid cert on the WebID provider. Presumably, it can be reproduced simply by having that. These are two very different bugs, but I was hoping there could be some generic fallback error handling...

  3. added
    inrupt-planningIssues that Inrupt will consider when planning development cycles
    and removed
    triageIssues that need team review
    on Nov 5, 2018
  4. added
    priority-highto be used for important issues and pull requests that need to be addressed soon
    on Nov 6, 2018
  5. added
    inrupt-sprintIssues that Inrupt plans to work on in near future
    and removed
    inrupt-planningIssues that Inrupt will consider when planning development cycles
    on Nov 14, 2018
  6. removed
    priority-highto be used for important issues and pull requests that need to be addressed soon
    on Nov 14, 2018
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

    buginrupt-sprintIssues that Inrupt plans to work on in near future

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions