Opened 4 weeks ago
Last modified 4 weeks ago
#35930 new Cleanup/optimization
Database password visible on debug page — at Version 2
Reported by: | bytej4ck | Owned by: | |
---|---|---|---|
Component: | Error reporting | Version: | dev |
Severity: | Normal | Keywords: | db, password, exposed |
Cc: | bytej4ck | Triage Stage: | Accepted |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
In debug page view, secrets are not visible due to masked with '*'. When there is mysql db connection error due to unreachable db server: self.connection = self.get_new_connection(conn_params) exposes db password under Local vars
dropdown.
conn_params {'charset': 'utf8mb4', 'client_flag': 2, 'conv': {0: <class 'decimal.Decimal'>, 1: <class 'int'>, 2: <class 'int'>, 3: <class 'int'>, 4: <class 'float'>, 5: <class 'float'>, 7: <function DateTime_or_None at 0x7f6218e5b490>, 8: <class 'int'>, 9: <class 'int'>, 10: <function Date_or_None at 0x7f6218e5b640>, 11: <function typecast_time at 0x7f6219d803a0>, 12: <function DateTime_or_None at 0x7f6218e5b490>, 13: <class 'int'>, 15: <class 'bytes'>, 245: <class 'bytes'>, 246: <class 'decimal.Decimal'>, 249: <class 'bytes'>, 250: <class 'bytes'>, 251: <class 'bytes'>, 252: <class 'bytes'>, 253: <class 'bytes'>, 254: <class 'bytes'>, <class 'array.array'>: <function array2Str at 0x7f6218e84160>, <class 'decimal.Decimal'>: <function Decimal2Literal at 0x7f6218e840d0>, <class 'datetime.date'>: <function Thing2Literal at 0x7f6218e84040>, <class 'datetime.datetime'>: <function DateTime2literal at 0x7f6218e5b6d0>, <class 'datetime.timedelta'>: <function DateTimeDelta2literal at 0x7f6218e5b760>, <class 'set'>: <function Set2Str at 0x7f6218e5bd90>, <class 'NoneType'>: <function None2NULL at 0x7f6218e5bf40>, <class 'int'>: <function Thing2Str at 0x7f6218e5be20>, <class 'float'>: <function Float2Str at 0x7f6218e5beb0>, <class 'bool'>: <function Bool2Str at 0x7f6218e5bc70>}, 'database': 'test-db', 'password': 'test_password', 'unix_socket': '/example/test-db', 'user': 'example_user'}
Would be better if all db credentials in debug mode should be masked also with '*'.
Change History (3)
by , 4 weeks ago
Attachment: | 2024-11-22_21-17.png added |
---|
comment:1 by , 4 weeks ago
Component: | Uncategorized → Error reporting |
---|---|
Resolution: | → needsinfo |
Status: | new → closed |
Type: | Uncategorized → Bug |
comment:2 by , 4 weeks ago
Description: | modified (diff) |
---|---|
Resolution: | needsinfo |
Status: | closed → new |
Summary: | Database password visible on debug page (view source only) → Database password visible on debug page |
Version: | 4.1 |
Note:
See TracTickets
for help on using tickets.
It's unclear how to reproduce the problem. Please reopen if you can provide a minimal example.