Skip to content

Commit 8b7f9c8

Browse files
Merge dashpay#6027: fix: actually use -socketevents
113b3fe fix: actually use `-socketevents` (UdjinM6) Pull request description: ## Issue being fixed or feature implemented dashpay#6007 follow-up ## What was done? ## How Has This Been Tested? check `socketevents` in `getnetworkinfo` response ## Breaking Changes n/a ## Checklist: - [x] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have added or updated relevant unit/integration/functional/e2e tests - [ ] I have made corresponding changes to the documentation - [x] I have assigned this pull request to a milestone _(for repository code-owners and collaborators only)_ ACKs for top commit: kwvg: ACK 113b3fe PastaPastaPasta: utACK 113b3fe Tree-SHA512: 50dcbdfe1f34e42e71078b585cfed2cd6b07f5f08c8296c7205367043e42e676c2eca47fa5193fdb9071eef202b01ba6e44ae2e3affb59a4e94196ecb6eb4350
2 parents 95c3532 + 113b3fe commit 8b7f9c8

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

src/init.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2516,9 +2516,11 @@ bool AppInitMain(const CoreContext& context, NodeContext& node, interfaces::Bloc
25162516
}
25172517

25182518
std::string sem_str = args.GetArg("-socketevents", DEFAULT_SOCKETEVENTS);
2519-
if (SEMFromString(sem_str) == SocketEventsMode::Unknown) {
2519+
const auto sem = SEMFromString(sem_str);
2520+
if (sem == SocketEventsMode::Unknown) {
25202521
return InitError(strprintf(_("Invalid -socketevents ('%s') specified. Only these modes are supported: %s"), sem_str, GetSupportedSocketEventsStr()));
25212522
}
2523+
connOptions.socketEventsMode = sem;
25222524

25232525
const std::string& i2psam_arg = args.GetArg("-i2psam", "");
25242526
if (!i2psam_arg.empty()) {

0 commit comments

Comments
 (0)