Skip to content

Conversation

@bradsaracik
Copy link

@bradsaracik bradsaracik commented Sep 30, 2025

unmerged PR #320 by @slonopotamus failed with authentication. This I think gets past it, but I only have run it on 4.27.2 under Linux (and in files using ubuntu24 to come in a separate PR)
Apologies this is only in Linux. maintainers allowed to edit
Black ran on file.

Note, this may also remove the need for https://github.com/adamrehn/ue4-docker/blob/master/test-suite/test-ue-releases.py#L28

# See https://forums.unrealengine.com/t/upcoming-disruption-of-service-impacting-unreal-engine-users-on-github/1155880
# In May 2023, Epics broke Commit.gitdeps.xml for *all existing releases up to 5.1.1* due to changes in their CDN
# we need to authenticate
password = os.getenv("GITPASS")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But where does GITPASS come from?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

@slonopotamus slonopotamus Sep 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, but that won't work on Windows...

@bradsaracik
Copy link
Author

ahhh might need a documentation piece to make this work. (sorry picking up from too long ago already)
the PAT personal access token used for EPIC/Unreal may have needed slightly extra permissions for this to work. If testing hits a 403 error, that will likely be it.
My token has read:org, repo, workflow, write:packages
but I'm no longer sure it needed all/exactly that. I might have been extra permissive

@bradsaracik bradsaracik mentioned this pull request Sep 30, 2025
@bradsaracik bradsaracik marked this pull request as ready for review October 2, 2025 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants