[SERVER-49417] sessionId use in "connection accepted" server log Created: 10/Jul/20  Updated: 29/Oct/23  Resolved: 04/Aug/20

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: 4.4.0-rc11
Fix Version/s: 4.4.1, 4.7.0

Type: Bug Priority: Major - P3
Reporter: Oleg Pudeyev (Inactive) Assignee: Billy Donahue
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Related
is related to SERVER-47020 Connection accepted and ended messages Closed
Backwards Compatibility: Fully Compatible
Operating System: ALL
Backport Requested:
v4.4
Participants:

 Description   

I see the following entries in my server log:

{"t":{"$date":"2020-07-09T22:36:12.727-04:00"},"s":"I",  "c":"NETWORK",  "id":22943,   "ctx":"listener","msg":"connection accepted","attr":{"remote":"127.0.0.1:37914","sessionId":9294,"connectionCount":9}}
{"t":{"$date":"2020-07-09T22:36:12.729-04:00"},"s":"I",  "c":"NETWORK",  "id":22943,   "ctx":"listener","msg":"connection accepted","attr":{"remote":"127.0.0.1:37916","sessionId":9295,"connectionCount":10}}
{"t":{"$date":"2020-07-09T22:36:12.730-04:00"},"s":"I",  "c":"NETWORK",  "id":22943,   "ctx":"listener","msg":"connection accepted","attr":{"remote":"127.0.0.1:37918","sessionId":9296,"connectionCount":11}}

By "sessionId" does the server mean "connectionId"? If so I expect the field to be called "connectionId".

  • Is this the same value as what ismaster returns under connectionId field?
  • Is this the same value which is later referenced as "conn9294" etc. in subsequent log entries?


 Comments   
Comment by Githook User [ 04/Aug/20 ]

Author:

{'name': 'Billy Donahue', 'email': 'billy.donahue@mongodb.com', 'username': 'BillyDonahue'}

Message: SERVER-49417 Fix connectionId attribute name

(cherry picked from commit 3310356893553ef42d838ad97ed8638e42cb6085)
Branch: v4.4
https://github.com/mongodb/mongo/commit/e7fbb9ff3ddc68d0b3362c0deddbf548442b84e6

Comment by Githook User [ 04/Aug/20 ]

Author:

{'name': 'Billy Donahue', 'email': 'billy.donahue@mongodb.com', 'username': 'BillyDonahue'}

Message: SERVER-49417 Fix connectionId attribute name
Branch: master
https://github.com/mongodb/mongo/commit/1d4e30220c8ca0ea002494d9df3f8ff9159e2096

Comment by Billy Donahue [ 30/Jul/20 ]

CR

https://mongodbcr.appspot.com/636800001/

Comment by Bruce Lucas (Inactive) [ 10/Jul/20 ]

Also since changing an attribute name is a mildly breaking change I would like to get this into 4.4.0 if possible else 4.4.1.

attn kelsey.schubert

Comment by Bruce Lucas (Inactive) [ 10/Jul/20 ]

When we fix this can we also please combine it with SERVER-47020 (log connectionId on connection ended message as well, capitalize "Connection accepted" and "Connection ended")?

Comment by Daniel Pasette (Inactive) [ 10/Jul/20 ]

Yeah, that looks like a mistake. I think the class name is misleading, but should be connectionId

https://github.com/mongodb/mongo/blob/master/src/mongo/transport/service_entry_point_impl.cpp#L179

Generated at Thu Feb 08 05:19:47 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.