Skip to content

test: Add known limitation workaround for accessing Twingate from docker container#36

Merged
ekampf merged 38 commits intomainfrom
fix/alpine-issue
Apr 28, 2025
Merged

test: Add known limitation workaround for accessing Twingate from docker container#36
ekampf merged 38 commits intomainfrom
fix/alpine-issue

Conversation

@ekampf
Copy link
Collaborator

@ekampf ekampf commented Apr 22, 2025

No description provided.

@ekampf ekampf changed the title DRAFT: test alpine access DRAFT: test secure access from a docker step Apr 22, 2025
@ekampf ekampf changed the title DRAFT: test secure access from a docker step test: Add known limitation workaround for accessing Twingate from docker container Apr 25, 2025
@ekampf ekampf requested a review from Copilot April 25, 2025 22:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation and a workaround for a known limitation when accessing Twingate from within a Docker container. Key changes include:

  • Adding a new “Known Limitations” section in README.md that explains how to override Docker's resolv.conf.
  • Updating the CI workflow to include steps for DNS resolution checking and access verification using Docker.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
README.md Adds documentation on the known limitation with Docker’s resolv.conf and provides a workaround.
.github/workflows/ci.yaml Adds new steps in the CI workflow to validate DNS resolution and access to secure resources.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ekampf ekampf marked this pull request as ready for review April 25, 2025 22:28
…ion of DNS configuration in CI workflow"

This reverts commit 2137e40.
@ekampf ekampf force-pushed the fix/alpine-issue branch from f01ea2c to d782ebf Compare April 25, 2025 23:25
@ekampf ekampf force-pushed the fix/alpine-issue branch from 9598d61 to 56cea5f Compare April 25, 2025 23:44
ekampf added 2 commits April 25, 2025 16:54
This reverts commit e656fbf.
@ekampf ekampf force-pushed the fix/alpine-issue branch 2 times, most recently from 3f708e7 to 2c8e9c6 Compare April 26, 2025 00:10
@ekampf ekampf force-pushed the fix/alpine-issue branch from 2c8e9c6 to c117bf7 Compare April 26, 2025 00:13
@ekampf ekampf force-pushed the fix/alpine-issue branch from 4030fa9 to c31d162 Compare April 26, 2025 00:19
Copy link
Contributor

@liorr liorr left a comment

Choose a reason for hiding this comment

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

F&M

@ekampf ekampf merged commit 88ceaad into main Apr 28, 2025
10 checks passed
@ekampf ekampf deleted the fix/alpine-issue branch April 28, 2025 19:27
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.

3 participants