https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40911

            Bug ID: 40911
           Summary: SIP Patron->new() generates ORM warnings with
                    undefined/empty patron_id
 Change sponsored?: ---
           Product: Koha
           Version: unspecified
          Hardware: All
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P5 - low
         Component: SIP2
          Assignee: [email protected]
          Reporter: [email protected]
        QA Contact: [email protected]

Noticed in production, when C4::SIP::ILS::Patron->new is passed a $patron_id
that is not defined or empty -or any processing makes it like that- (e.g.
faulty plugins, malformed SIP requests) logs get flooded with DBIx::Class
errors and Perl warnings when it should never reach the ORM with a query
because there's nothing to search for.

Similar to bug 40910 for Item->new(), the Patron constructor should validate
input early and return gracefully without generating warnings for
undefined/empty patron  identifiers.

Expected behavior: Silent return for invalid input, proper debug logging for
troubleshooting
Actual behavior: Warning spam in SIP server logs, potential DBIx::Class
constraint errors

This affects SIP server stability and makes log analysis difficult due to noise
from invalid requests.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to