common-fate / granted

The easiest way to access your cloud.
https://granted.dev
MIT License
957 stars 90 forks source link

Fix for registry sync issue #587 #588

Closed shwethaumashanker closed 6 months ago

shwethaumashanker commented 6 months ago

What changed?

The copyFile function in sync.go had a mistake in the order of arguments when calling io.Copy. The io.Copy function takes the destination writer as the first argument and the source reader as the second argument.

Why?

Since this was misconfigured the profiles outside of the registry from ~/.aws/config where not being copied

How did you test it?

dgranted registry sync locally

Potential risks

Is patch release candidate?

Link to relevant docs PRs