Uploaded image for project: 'FOLIO'
  1. FOLIO
  2. FOLIO-3149

vertx-pac4j: form parameters not read correctly

    XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • TBD
    • Resolution: Done
    • None
    • None
    • CP: sprint 113
    • 1
    • Core: Platform

    Description

      MODLOGSAML-97 reveals that vertx-pac4j 5.0.2 (FOLIO) does not read form parameters

      We are maintaing:

      https://github.com/folio-org/vertx-pac4j/commits/vertx-4-0-2-folio

      (branch is named after Vert.x, not pac4j version, which admittedly, could be confusing)

      Let's fix it here:
      https://github.com/folio-org/vertx-pac4j/blob/2f3c3cbb3ce86ff080ed401e9d645881e784d62b/src/main/java/org/pac4j/vertx/VertxWebContext.java#L82

      and release this. At the same time, vertx-pac4j might as well be upgraded to use 4.0.3

      TestRail: Results

        Attachments

          Issue Links

            blocks

            Bug - A problem which impairs or prevents the functions of the product. MODLOGSAML-97 Single-Sign-On (SSO) always fails

            • P1 - highest priority item, drop everything else before this is resolved, reserved for critical bugfixes
            • Closed
            is duplicated by

            Bug - A problem which impairs or prevents the functions of the product. FOLIO-3148 Form parameters not inspected in vertx-pac4j

            • TBD - To be determined
            • Closed
            relates to

            Bug - A problem which impairs or prevents the functions of the product. FOLIO-3153 UPSTREAM: vertx-pac4j: form parameters not read correctly - upstream

            • P3 - low priority level, item will be considered for inclusion in the next dev cycle
            • Closed

            Task - A task that needs to be done. FOLIO-3150 Release vertx-pac4j 5.0.2-FOLIO.1

            • TBD - To be determined
            • Closed

            Task - A task that needs to be done. FOLIO-3333 Release vertx-pac4j 6.0.0 FOLIO

            • TBD - To be determined
            • Closed

            Activity

              People

                adam Adam Dickmeiss (Inactive)
                adam Adam Dickmeiss (Inactive)
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  TestRail: Runs

                    TestRail: Cases