• FreedomAdvocate@lemmy.net.au
    link
    fedilink
    English
    arrow-up
    2
    arrow-down
    2
    ·
    5 days ago

    The issue they had was being marked as phishing, not the SSL certificate warning page.

    Have you seen what browsers say when you have a look at the SSL certificate warning page?

    It is for pull requests. A user makes a change to the documentation, they want to be able to see the changes on a web page.

    Why is a user made PR publishing a branch to Immich’s domain for the user to see?

    • BCsven@lemmy.ca
      link
      fedilink
      English
      arrow-up
      1
      ·
      4 days ago

      I thought that was how pull requests worked, its a branch if you’veade a departure to edit code, you have the pull request and ask them to merge into the main branch. It should be visible to everyone so everyone can review the change.

      • FreedomAdvocate@lemmy.net.au
        link
        fedilink
        English
        arrow-up
        1
        ·
        12 hours ago

        The branch for the PR shouldn’t be hosted on the production site’s domain, and that deployment that the company will be testing and reviewing shouldn’t be accessible to the public. They even have internal in the URL, while being accessible by external people lol