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

quic: multiple cleanups #34137

Closed
wants to merge 8 commits into from
Closed

quic: multiple cleanups #34137

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
a1b9e76
quic: cleanups in JS API
jasnell Jun 27, 2020
e52f81d
quic: cleanups for QuicSocket
jasnell Jun 30, 2020
f3ead5d
quic: fixup set_final_size
jasnell Jun 30, 2020
0aa82a4
quic: consolidate onSessionClose and onSessionSilentClose
jasnell Jun 30, 2020
e6f6ff1
quic: remove unused callback function
jasnell Jun 30, 2020
b1c13b8
quic: fixup constant exports, export all protocol error codes
jasnell Jun 30, 2020
4135c5c
quic: avoid using private JS fields for now
jasnell Jun 30, 2020
0c9ea4e
quic: reduce duplication of code
jasnell Jun 30, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
quic: remove unused callback function
  • Loading branch information
@jasnell
jasnell committed Jun 30, 2020
commit e6f6ff179eaf90e4228a46fae4857f5125a470ed
9 changes: 0 additions & 9 deletions lib/internal/quic/core.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -474,14 +474,6 @@ function onSessionQlog(str) {
}
}

// Called when an error occurs in a QuicSession. When this happens,
// the only remedy is to destroy the session.
function onSessionError(error) {
if (this[owner_symbol]) {
this[owner_symbol].destroy(error);
}
}

// Called by the C++ internals when a client QuicSession receives
// a version negotiation response from the server.
function onSessionVersionNegotiation(
Expand Down Expand Up @@ -570,7 +562,6 @@ setCallbacks({
onSessionClientHello,
onSessionClose,
onSessionDestroyed,
onSessionError,
onSessionHandshake,
onSessionKeylog,
onSessionQlog,
Expand Down
1 change: 0 additions & 1 deletion src/env.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -455,7 +455,6 @@ constexpr size_t kFsStatsBufferLength =
V(quic_on_session_client_hello_function, v8::Function) \
V(quic_on_session_close_function, v8::Function) \
V(quic_on_session_destroyed_function, v8::Function) \
V(quic_on_session_error_function, v8::Function) \
V(quic_on_session_handshake_function, v8::Function) \
V(quic_on_session_keylog_function, v8::Function) \
V(quic_on_session_path_validation_function, v8::Function) \
Expand Down
1 change: 0 additions & 1 deletion src/quic/node_quic.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,6 @@ void QuicSetCallbacks(const FunctionCallbackInfo<Value>& args) {
SETFUNCTION("onSessionClientHello", session_client_hello);
SETFUNCTION("onSessionClose", session_close);
SETFUNCTION("onSessionDestroyed", session_destroyed);
SETFUNCTION("onSessionError", session_error);
SETFUNCTION("onSessionHandshake", session_handshake);
SETFUNCTION("onSessionKeylog", session_keylog);
SETFUNCTION("onSessionUsePreferredAddress", session_use_preferred_address);
Expand Down