connect icon indicating copy to clipboard operation
connect copied to clipboard

Added ports back to the Connect Sync container

Open erik-pineiro-github opened this issue 3 years ago • 1 comments

After confirming with @edif2008, the Connect Sync container should also have ports mapped per the example found in the developer documentation.

erik-pineiro-github avatar Sep 27 '22 18:09 erik-pineiro-github

Hey @erik-pineiro-github, thanks for filing the PR. In my mind we should attack this by linking the example file in the documentation to this GitHub repository rather than changing the port designation here.

Just to clarify, why is this port needing to be exposed on the Docker host? The connect-sync container is responsible for connecting to 1Password.com to sync vault data, and should not require any ingress whatsoever.

ag-adampike avatar Sep 27 '22 20:09 ag-adampike