Skip to content

Conversation

@jglogan
Copy link
Contributor

@jglogan jglogan commented Oct 26, 2025

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Testing

  • Tested locally
  • Added/updated tests
  • Added/updated docs

if let socketUrl = Self.hostSocketUrl(config: config) {
if let socketUrl = Self.sshAuthSocketHostUrl(config: config) {
let socketPath = socketUrl.path(percentEncoded: false)
let attrs = try FileManager.default.attributesOfItem(atPath: socketPath)
Copy link
Contributor

Choose a reason for hiding this comment

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

do we wanna handle the scenario when attributesOfItem throws?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah yeah, let's default to what we were doing before for that case as well

@realrajaryan realrajaryan merged commit a23bcf0 into apple:main Oct 27, 2025
2 checks passed
saehejkang pushed a commit to saehejkang/container that referenced this pull request Oct 28, 2025
…e#811)

- Closes apple#580.
- Depends on apple/containerization#362.

## Type of Change
- [x] Bug fix
- [ ] New feature  
- [ ] Breaking change
- [ ] Documentation update

## Testing
- [x] Tested locally
- [ ] Added/updated tests
- [ ] Added/updated docs
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.

[Bug]: --ssh flag doesn't work with non-root user

2 participants