Project

General

Profile

Actions
Copy link

Bug #8258

closed
GL GL

ldap: no invalid_data event in case of invalid request

Bug #8258: ldap: no invalid_data event in case of invalid request

Added by Giuseppe Longo 2 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Assignee:
Giuseppe Longo
Target version:
9.0.0-beta1
Affected Versions:
git main
Effort:
Difficulty:
Label:

Description

While reviewing the LDAP code, I noticed that in parse_request:

1. tx.tx_data.updated_ts is not set to true when a new transaction is created.
2. LdapEvent::InvalidData is not set when a request is not parsed correctly.

Subtasks 1 (0 open — 1 closed)

Bug #8317: ldap: no invalid_data event in case of invalid request (8.0.x backport)ClosedGiuseppe LongoActions

VJ Updated by Victor Julien 2 months ago Actions
Copy link
 #1

  • Status changed from New to In Progress
  • Target version changed from TBD to 9.0.0-beta1
  • Affected Versions git main added
  • Affected Versions deleted (9.0.0-beta1)

Do we need a backport to 8 as well? Is there a test?

PA Updated by Philippe Antoine 2 months ago Actions
Copy link
 #2

  • Subject changed from ldap: set tx_data.updated_ts and invalid_data event to ldap: set invalid_data event

1. tx.tx_data.updated_ts is not set to true when a new transaction is created.

For me this statement is wrong, tx.tx_data.updated_ts is set to true when a new transaction is created.

2. LdapEvent::InvalidData is not set when a request is not parsed correctly.

There is a missed case here indeed

PA Updated by Philippe Antoine 2 months ago Actions
Copy link
 #3

  • Subject changed from ldap: set invalid_data event to ldap: set invalid_data event in all cases

GL Updated by Giuseppe Longo about 2 months ago Actions
Copy link
 #4

  • Target version changed from 9.0.0-beta1 to 8.0.4

VJ Updated by Victor Julien about 2 months ago Actions
Copy link
 #5

  • Target version changed from 8.0.4 to 9.0.0-beta1
  • Label Needs backport to 8.0 added

OT Updated by OISF Ticketbot about 2 months ago Actions
Copy link
 #6

  • Subtask #8317 added

OT Updated by OISF Ticketbot about 2 months ago Actions
Copy link
 #7

  • Label deleted (Needs backport to 8.0)

SB Updated by Shivani Bhardwaj about 1 month ago Actions
Copy link
 #8

  • Subject changed from ldap: set invalid_data event in all cases to ldap: no invalid_data event in case of invalid request

PA Updated by Philippe Antoine about 1 month ago Actions
Copy link
 #9

  • Status changed from In Progress to Resolved

PA Updated by Philippe Antoine about 1 month ago Actions
Copy link
 #10

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: PDF Atom