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

test: tests for #703 #704

Closed
wants to merge 75 commits into from
Closed

test: tests for #703 #704

wants to merge 75 commits into from

Conversation

XavierChanth
Copy link
Member

@XavierChanth XavierChanth commented Jan 22, 2024

- What I did

See #703

#703 is a stack on top of #659

This PR exists to ensure that the tests still pass for the changes in #703

- How I did it

- How to verify it

- Description for the changelog
test: tests for #703

gkc and others added 30 commits November 23, 2023 12:19
docs: Added TODOs
…esult of many a files uptaking the new name.
…-option

# Conflicts:
#	packages/noports_core/lib/src/sshnp/impl/sshnp_openssh_local_impl.dart
#	packages/noports_core/lib/src/sshrvd/sshrvd_impl.dart
1. Changes to take args to accept authenticateClient and authenticateDevice args
2. Using the values of these args to make decisions
3. Changes to the way we sign the data
- new sshrv parameters --bind-local-port and --rvd-auth-string
- uses new socket_connector method, serverToSocket
- multiple other tweaks to get things to work
…d IV in sshrv. Next commit will have the AES key and IV creation, sharing, and passing in to the sshrv process via command line and parameters
- add checkForFinalDeliveryStatus and waitForFinalDeliveryStatus parameters everywhere we call `notify`
- make callers use those parameters appropriately.
Note that in no case was there a need for those parameters to be true, but it seems better to introduce the parameters and make callers explicitly aware of them, rather than the simpler change to just set them to false in every case.
…s the daemon supports. This is then used to downgrade where the daemon does not support the feature
…ted out two 'return' statements for now, in the runZonedGuarded onError handler
gkc and others added 26 commits January 9, 2024 11:31
chore: socket-authenticator-option merge renames
…-option

# Conflicts:
#	packages/dart/noports_core/lib/src/sshnp/impl/sshnp_openssh_local_impl.dart
#	packages/dart/noports_core/lib/src/version.dart
#	packages/dart/noports_core/pubspec.yaml
#	packages/dart/sshnoports/pubspec.lock
#	packages/dart/sshnoports/pubspec.yaml
1. have SshrvdImpl fetch the public key of another atSign via the Sshrvd's own atClient
2. instead of waiting 1 second for a forked sshrv process to have started up, instead listen to its stderr and wait for the message that says "rv is running"
…Rvd, encryptRvdTraffic and discoverDaemonFeatures to false
- if new rvd features (authentication) are NOT being used, send an old-style request
- if new rvd features (authentication) ARE being used, send a new-style request
…g because of timeout waiting for sshrvd response
- in ping response, daemon says whether or not it accepts public keys from the client
- added abbreviations in SshnpArg for the new flags
- corrected some code comments and added to others
@XavierChanth
Copy link
Member Author

XavierChanth commented Jan 22, 2024

Same recurrent issue where local-trunk and trunk-local failed because I rename something in the e2e test suite

@XavierChanth
Copy link
Member Author

nvm #703 is running its own set of tests

@gkc gkc deleted the rename-sr branch January 22, 2024 11:30
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