SIP filters

Bug #1989481 reported by Jason Etheridge
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
SIPServer
New
Undecided
Unassigned

Bug Description

Depends on bug 1989478

squashed: https://git.evergreen-ils.org/?p=working/SIPServer.git;a=shortlog;h=refs/heads/collab/phasefx/sip-redaction-squashed
more granular scratchwork: https://git.evergreen-ils.org/?p=working/SIPServer.git;a=shortlog;h=refs/heads/collab/phasefx/sip-redaction-squashed

This is new functionality that allows you to add stanzas like these to the SIP configuration file for modifying SIP responses:

<filters>
    <!--
        Filters allow SIPServer to strip or modify specified fields inline from outgoing SIP messages. Currently, these
        rules apply to all message types and only work for variable-length fields, but the config layout leaves room for
        future enhancement.

        Examples:

        <field identifier="AE" replace_with="John Doe" />
        <field identifier="BE" remove="true" />
    -->
</filters>

Not yet pullrequesting until we're happy with the SIP test suite revitalization branch.

Tags: pullrequest
Revision history for this message
Jason Etheridge (phasefx) wrote :

I removed the dependency on bug 1989478 and cherry-picked the commit into master and resolved a minor conflict. The test suite as a whole still doesn't pass, but 16filters.t does, assuming a concerto database by default, or by modifying SIPtest.pm as appropriate (which should be reconciled with oils_sip.xml anyway when testing).

https://git.evergreen-ils.org/?p=working/SIPServer.git;a=shortlog;h=refs/heads/collab/phasefx/sip-filters-on-master

So... pullrequested. :-)

tags: added: pullrequest
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.