Uploaded image for project: 'folio-vertx-lib'
  1. folio-vertx-lib
  2. VERTXLIB-14

Allow _ as part of tenant id

    XMLWordPrintable

Details

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

    Description

      Currently _ is not allowed to be part of tenant id.
      https://github.com/folio-org/folio-vertx-lib/blob/2fa365266cb77cac2429bc23e456440e80d94fdd/core/src/main/java/org/folio/tlib/util/TenantUtil.java#L11

      Apparently RMB and existing modules have allowed the use of underscore. The problem with underscore in this context is that could, under very special circumstances, be mixed with module name.

      Say we have module called "users" and another module called "mod-users" (normalized to mod_users).

      Tenant diku_mod and module users would use schema diku_mod_users.

      Tenant diku and module mod-users would use schema diku_mod_users.

      Attachments

        Issue Links

          blocks

          Bug - A problem which impairs or prevents the functions of the product. MODEUR-130 tenant id including _ is rejected

          • TBD - To be determined
          • Closed
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. OKAPI-1081 Reject invalid tenant ids

          • P2 - normal priority level, must be included in the current development cycle
          • Blocked

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. RMB-902 Reject tenant with invalid characters and possibly sanitize

          • TBD - To be determined
          • Open

          Activity

            People

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

              Dates

                Created:
                Updated:
                Resolved: