Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #367: Git Private Repository Authentication #368

Closed
wants to merge 2 commits into from

Conversation

seandenigris
Copy link
Collaborator

Respect siteUsername: and sitePassword when downloading Zips.
OT: Fix "Metacello-Platform" package pharo30 branch, which had incorrect meta-data incorrectly specifying the package name including the branch i.e. "Metacello-Platform.pharo30"

Sean DeNigris added 2 commits August 21, 2015 19:45
- For compatibility with platforms besides recent Pharo, default to old-style un-authenticated #downloadZipArchive:to:; ideally, someone would implement #downloadZipArchive:to:username:pass: on those other platforms
@seandenigris
Copy link
Collaborator Author

What needs to be done here?

@dalehenrich
Copy link
Member

Given that there were a large number of test failures one step is to rerun the tests, but at the moment I am struggling with getting Squeak4.6 and Squeak5.0 added so I've prioritized doing that inform of this one ....

I haven't looked at all of the failing tests, so I don't know if there are additional issues or not ... the load failures are due to network failures and rerunning should show whether or there are actual code-based problems ...

@dalehenrich
Copy link
Member

@seandenigris, when you get a chance can you make a minor change to a file (like add a line to the README.md)? I'm getting massive failures for tests and I'm not sure whether travis is just confused or something else is going on ... "pinging travis" will at least eliminate travis is confused ...

@davelab6
Copy link

I guess this should be closed

@dalehenrich
Copy link
Member

Closing this PR - storing username/passwords in the image is not a good idea. the discussion in Issue #475 addresses the larger issue of enabling "first class access" to private repositories that require usernames and passwords ...

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.

None yet

3 participants