Researchers at Truffle Security have found, or arguably rediscovered, that data from deleted GitHub repositories (public or private) and from deleted copies (forks) of repositories isn’t necessarily deleted.

Joe Leon, a security researcher with the outfit, said in an advisory on Wednesday that being able to access deleted repo data – such as APIs keys – represents a security risk. And he proposed a new term to describe the alleged vulnerability: Cross Fork Object Reference (CFOR).

“A CFOR vulnerability occurs when one repository fork can access sensitive data from another fork (including data from private and deleted forks),” Leon explained.

For example, the firm showed how one can fork a repository, commit data to it, delete the fork, and then access the supposedly deleted commit data via the original repository.

The researchers also created a repo, forked it, and showed how data not synced with the fork continues to be accessible through the fork after the original repo is deleted. You can watch that particular demo.

  • @AdamEatsAss
    link
    English
    535 months ago

    Oh god. That means all the spaghetti code that I ever wrote is still out there.

    • @[email protected]
      link
      fedilink
      English
      215 months ago

      Yup. Along with the code from huge organizations. I always thought it was funny that people put their code online, blindly trusting some random company that got gobbled up by Microsoft.

        • @[email protected]
          link
          fedilink
          English
          115 months ago

          Ha ha, way way back in the day when I didn’t understand how keys worked, I sent a private key to another developer when they asked for my public. They were kind enough to educate me.

          • @[email protected]
            link
            fedilink
            English
            -35 months ago

            As a lifelong troll, I would’ve just generated a new pub key and made a bunch of commits as you. Then two days later, I would tell you what’s up once you had time to process the confusion.

      • @Chocrates
        link
        English
        85 months ago

        Your point is valid, but many (most?) enterprises don’t use a forking worlflow, so I suspect open source projects will be hit harder, sadly

    • @Cosmos7349
      link
      English
      105 months ago

      Not only just out there. I am regenerating your spaghetti code into a new context with copilot 🧑‍✈️ Your (ai-regenerated) code will be driving our military nuclear launch code base! Congratulations!

  • Mubelotix
    link
    fedilink
    English
    335 months ago

    This is not a GitHub issue. It’s a GIT feature. People are always going to clone your repo.

    • Morphit
      link
      fedilink
      English
      85 months ago

      Well, sort of. GitHub certainly could refuse to render orphan commits. They pop up a banner saying so but I don’t see why they should show the commit at all. They could still keep the data until it’s garbage collected since a user might re-upload the commit in a new branch.

      This seems like a non-issue though since someone who hasn’t already seen the disclosed information would need to somehow determine the hash of the deleted commit.

      • Morphit
        link
        fedilink
        English
        85 months ago

        Ah - Actually reading the article reveals why this is actually an issue:

        What’s more, Ayrey explained, you don’t even need the full identifying hash to access the commit. “If you know the first four characters of the identifier, GitHub will almost auto-complete the rest of the identifier for you,” he said, noting that with just sixty-five thousand possible combinations for those characters, that’s a small enough number to test all the possibilities.

        So enumerating all the orphan commits wouldn’t be that hard.

        In any case if a secret has been publicly disclosed, you should always assume it’s still out there. For sure, rotate your keys.

    • @[email protected]
      link
      fedilink
      English
      75 months ago

      Forks do not exist in git. It’s a GitHub feature, and a massive blunder at the same time.

        • @arcuru
          link
          English
          75 months ago

          The article is specifically about how GitHub forks are not the same as a git clone. A clone isn’t accessible from the upstream without the upstream pulling the changes, but this vulnerability points out that a fork on GitHub is accessible from the upstream without a pull, even if the fork is private.

          It’s because GitHub under the hood doesn’t actually do a real clone so that they can save on disk usage.

          • Mubelotix
            link
            fedilink
            English
            05 months ago

            You actually can’t turn a fork private on github

        • @[email protected]
          link
          fedilink
          English
          45 months ago

          How can such a wrong answer get so many points? Clones and forge forks are unrelated. First, GitHub or GitLab cannot and could not link clones together without analyzing the remotes of each clone.

          FFS it’s a tech community…

          • Mubelotix
            link
            fedilink
            English
            -35 months ago

            Because you are the one being wrong. Github and others only provide a nice interface around clones. That’s all there is, and it doesn’t matter much

  • @kernelle
    link
    English
    115 months ago

    So many OpenAI keys!

  • @[email protected]
    link
    fedilink
    English
    -85 months ago

    Classic microsoft. Use other git instances please. If you want actions you can use any public Forejo instance.

    • Eager Eagle
      link
      English
      205 months ago

      that’s a direct cause of how forks work, it most likely predates microsoft’s acquisition

          • Mubelotix
            link
            fedilink
            English
            5
            edit-2
            5 months ago

            Imagine creating a whole new “universal” language and using such shitty diacritics that nobody likes

            • @[email protected]
              link
              fedilink
              English
              15 months ago

              Yeah, kinda dumb. But they do have a relatively popular workaround: the x-system. So forĝejo becomes forgxejo (x = diacritic for the prev letter).