checking for req.socket.servername on plain sockets #18

Closed
opened 6 years ago by coolaj86 · 1 comments
coolaj86 commented 6 years ago
Owner

Should first check for req.socket.encrypted

        } else if (safehost && !gl.middleware.sanitizeHost._skip_fronting_check) {
          // TODO how to handle wrapped sockets, as with telebit?
          console.warn(safehost, typeof req.socket.servername, req.servername, req.socket.servername, req.socket.encrypted);
          console.warn("\n\n\n[greenlock] WARN: no string for req.socket.servername,"
            + " skipping fronting check for '" + safehost + "'\n\n\n");
Should first check for req.socket.encrypted ``` } else if (safehost && !gl.middleware.sanitizeHost._skip_fronting_check) { // TODO how to handle wrapped sockets, as with telebit? console.warn(safehost, typeof req.socket.servername, req.servername, req.socket.servername, req.socket.encrypted); console.warn("\n\n\n[greenlock] WARN: no string for req.socket.servername," + " skipping fronting check for '" + safehost + "'\n\n\n"); ```
Poster
Owner

Fixed in v2.4.8

Fixed in v2.4.8
coolaj86 closed this issue 6 years ago
coolaj86 referenced this issue from a commit 5 years ago
Sign in to join this conversation.
No Label
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.