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

Update README to include Poetry system Git client configuration workaround #237

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TinyMarsh
Copy link

This change addresses a HangupException error that occurs when Poetry's default Git client fails to work with the SSH configuration provided by this action. The solution ensures compatibility by switching to the system-installed Git client.

Relevant chunk of the stack trace below. This only seemed to be an issue on Windows runners:

  - Installing <private_repo> (0.1.0 68fd337)

  HangupException

  The remote server unexpectedly closed the connection.

  at C:\Program Files (x86)\pipx\venvs\poetry\lib\site-packages\dulwich\protocol.py:255 in read_pkt_line
      251| 
      252|         try:
      253|             sizestr = read(4)
      254|             if not sizestr:
    > 255|                 raise HangupException
      256|             size = int(sizestr, 16)
      257|             if size == 0 or size == 1:  # flush-pkt or delim-pkt
      258|                 if self.report_activity:
      259|                     self.report_activity(4, "read")

Cannot install <private_repo>.

Error: Process completed with exit code 1.

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.

1 participant