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.

Sign up for GitHub

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

Jump to bottom

GH-499: Automatic processing of IP changes #335

Open
wants to merge 388 commits into
base: master
Choose a base branch
from
Open

GH-499: Automatic processing of IP changes #335

wants to merge 388 commits into from

Conversation

dnwiebe
Copy link
Collaborator

@dnwiebe dnwiebe commented Aug 28, 2023

No description provided.

@dnwiebe
GH-372: Interim commit
d9fb506
@dnwiebe
GH-372: Interim commit
8213a1e
@dnwiebe
GH-372: Interim commit
69f9006
@dnwiebe
GH-372: Another interim commit
3b99080
@dnwiebe
GH-372: PCP seems to be working
956255d
@dnwiebe
Interim commit
94b92ce
@dnwiebe
GH-327: Interim commit
53fc364
@dnwiebe
GH-327: PMP compiles now, doesn't pass
c5a54cc
@dnwiebe
GH-372: First successful change-handler test for PMP
cd0f03d
@dnwiebe
GH-372: Clippy appeasement
e7f72a2
@dnwiebe
GH-372: find_working_protocol is working for one protocol
1672652
@dnwiebe
GH-372: Interim commit
1dd03b7
@dnwiebe
GH-372: End-of-morning commit
8454af4
@dnwiebe
GH-372: New architecture code is working
dc206b6
@dnwiebe
GH-382: Implemented usual_protocol_opt
34ebf15
@dnwiebe
GH-372: Finished get_public_ip, working on add_mapping
3c54016
@dnwiebe
GH-372: I think I'm going to try modifying the Experiment definition
879a8d6
@dnwiebe
GH-372: Now to test-drive some failures
5681831
@dnwiebe
GH-372: Friday morning's work
5bee2be
@dnwiebe
GH-372: Extended the Daemon's Node-startup timeout
4a994c9
@dnwiebe
GH-372: AutomapControl seems to be done
72f1706
@dnwiebe
GH-372: IGDP tests pass
e1a16d5
@dnwiebe
GH-372: Removed warnings; three tests failing
b7a4adf
@dnwiebe
GH-372: start_change_handler should work for everybody now
b49fbb1
@dnwiebe
GH-372: Formatting and high-level cleanup
643dcfe
@dnwiebe
GH-372: Two todos left in pmp.rs; one test failing
1c5d677
@dnwiebe
GH-372: PMP free of TODOs and tests passing
2c573e1
@dnwiebe
GH-372: TODOs gone from IGDP
30eabac
@dnwiebe
GH-372: All TODOs gone from Automap
f0e7b30
@dnwiebe
GH-372: Node unit tests pass
7a466e2
dnwiebe added 30 commits July 2, 2023 14:48
@dnwiebe
Tests are passing
77ac4ce
@dnwiebe
Just a point to come back to if everything turns to garbage
6700c7a
@dnwiebe
Pulled change_handler back out of AutomapConfig in defeat
9a845ae
@dnwiebe
Finally got handle_automap_error_delegates_correctly_to_factory prope…
abf55cb 
…rly asserted
@dnwiebe
Decided not to test AutomapControlFactory since it's part of the Fact…
9568df7 
…ory pattern
@dnwiebe
Review issues
e03a8fa
@dnwiebe
Review issues
574e487
@dnwiebe
Review issues
b3c4b42
@dnwiebe
Merge branch 'master' into GH-499-merge
e63c260
@dnwiebe
Partway through merge of master
1dd81f3
@dnwiebe
All Node/node tests passing now
200452c
@dnwiebe
All non-multinode tests pass
49e38af
@dnwiebe
All tests passing except that pesky multinode IP-change test
193b80f
@dnwiebe
Ignoring the IpChange tests for Actions
40b3243
@dnwiebe
Mock router compiles but doesn't work
8035ee2
@dnwiebe
See if this makes it through Actions
9198ed0
@dnwiebe
Correcting Windows-specific errors
7b14399
@dnwiebe
Formatting
b000fc1
@dnwiebe
Maybe windows_specific.rs is fixed.
0b2d8e8
@dnwiebe
Formatting...(blush)
42cb899
@dnwiebe
Fixed the macOS-specific code, hopefully
7485366
@dnwiebe
Formatting...
e92428b
@dnwiebe
MASQNodeUtils -> DataProbeUtils
e595e15
@dnwiebe
Fixed multinode_integration_tests
74cc42a
@dnwiebe
Experiment
8b86f50
@dnwiebe
Corrected one of the tests failing in Windows, hopefully
680d77c
@dnwiebe
Windows _does_ use that constant!
39ee99a
@dnwiebe
Added special-squirrel Windows error value
981a1c7
@dnwiebe
Modified Windows test to correspond with its name
a099731
@dnwiebe
Adjusted expectation in Windows-only test
03bf741
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dnwiebe @bertllll