Skip to content

Commit c53b2df

Browse files
author
dave
committed
Minor change to ensure connection is reset properly after authentication failure.
1 parent 677d5e2 commit c53b2df

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/RemoteConnector.cpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ void TagValueRemoteConnector::setRemoteConnected(uint8_t major, uint8_t minor, A
7979
}
8080
else {
8181
serdebugF("Not authenticated, dropping");
82-
transport->close();
82+
close();
8383
}
8484
}
8585

@@ -90,7 +90,7 @@ void TagValueRemoteConnector::provideAuthentication(const char* auth) {
9090
encodeAcknowledgement(0, ACK_CREDENTIALS_INVALID);
9191
taskManager.yieldForMicros(15000);
9292
setAuthenticated(false);
93-
transport->close();
93+
close();
9494
commsNotify(COMMSERR_DISCONNECTED);
9595
}
9696
else {
@@ -108,10 +108,12 @@ const char* lastUuid;
108108
void onPairingFinished(ButtonType ty, void* voidConnector) {
109109
auto* connector = reinterpret_cast<TagValueRemoteConnector*>(voidConnector);
110110
if(ty==BTNTYPE_ACCEPT) {
111+
serdebugF("Adding key after user pressed accept");
111112
bool added = connector->getAuthManager()->addAdditionalUUIDKey(connector->getRemoteName(), lastUuid);
112113
connector->encodeAcknowledgement(0, added ? ACK_SUCCESS : ACK_CREDENTIALS_INVALID);
113114
}
114115
else {
116+
serdebugF("Not adding key, close pressed");
115117
connector->encodeAcknowledgement(0, ACK_CREDENTIALS_INVALID);
116118
}
117119
}

0 commit comments

Comments
 (0)