Uploaded image for project: 'Sentry (Retired)'
  1. Sentry (Retired)
  2. SENTRY-380

Clean up some grantorPrincipal semantics

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.4.0
    • 1.5.0
    • None
    • None

    Description

      Came from SENTRY-327 review:

      • I do not think grantorPrincipal is required for createRole, we should instead pass it for alterSentryRoleGrantPrivilege and alterSentryRoleRevokePrivilege?
      • As we pass grantorPrincipal as the user invoking the thrift request(requestorUserName), we may keep it consistent and avoid possibility of passing different requestorUserName and grantorPrincipal by getting rid of grantorPrincipal field in the TSentryPrivilege and TSentryRole?

      Attachments

        1. SENTRY-380.2.patch
          91 kB
          Sravya Tirukkovalur
        2. SENTRY-380.1.patch
          90 kB
          Sravya Tirukkovalur
        3. SENTRY-380.0.patch
          81 kB
          Sravya Tirukkovalur

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. SENTRY-448 Update "sentry_policy_service.thrift" after clean up grantorPrincipal

          • Major - Major loss of function.
          • Resolved
          relates to

          Bug - A problem which impairs or prevents the functions of the product. SENTRY-444 Update the schema upgrade scripts per the grantor principal changes

          • Major - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. SENTRY-447 Fix thrift generated code related to grantor principal cleanup

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. SENTRY-440 Update jdo and object model to store grantorPrincipal in the mappings

          • Major - Major loss of function.
          • Open
          links to

          Web Link Review board link

          Activity

            People

              sravya Sravya Tirukkovalur
              sravya Sravya Tirukkovalur
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: