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

Add integration tests #25

Merged
merged 1 commit into from
Mar 26, 2024
Merged

Add integration tests #25

merged 1 commit into from
Mar 26, 2024

Conversation

wiktor-k
Copy link
Owner

@wiktor-k wiktor-k commented Mar 26, 2024

This PR addresses the problem of checking if the agent is properly functioning for it's most basic task: signing requests.

Instead of spawning a new sshd daemon this PR takes a different approach: using SSH file signatures which execute the same piece of code.

On Windows a slight change is required: a file is created to signal to the script that it's ready to serve requests. Sadly it doesn't seem that checking IF EXISTS "\\.\pipe\agent" works :/

@wiktor-k wiktor-k force-pushed the add-e2e-tests branch 17 times, most recently from 8a94705 to b4f6d9b Compare March 26, 2024 14:03
This PR addresses the problem of checking if the agent is properly
functioning for it's most basic task: signing requests.

Instead of spawning a new sshd daemon this PR takes a different approach:
using SSH file signatures which execute the same piece of code.

On Windows a slight change is required: a file is created to signal to
the script that it's ready to serve requests. Sadly it doesn't seem that
checking `IF EXISTS "\\.\pipe\agent"` works.

Fixes: #12
Signed-off-by: Wiktor Kwapisiewicz <[email protected]>
@wiktor-k
Copy link
Owner Author

I'm going to merge this as it's going to be critically important in #26.

@wiktor-k wiktor-k merged commit 3263c3d into main Mar 26, 2024
15 checks passed
@wiktor-k wiktor-k deleted the add-e2e-tests branch March 26, 2024 14:50
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