Inedo Community Forums Forums
    • Recent
    • Tags
    • Popular
    • Login

    Welcome to the Inedo Forums! Check out the Forums Guide for help getting started.

    If you are experiencing any issues with the forum software, please visit the Contact Form on our website and let us know!

    ProGet Entra Hybrid Authentication

    Scheduled Pinned Locked Moved Support
    13 Posts 3 Posters 48 Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • stevedennisS Offline
      stevedennis inedo-engineer @yaakov.smith_7984
      last edited by

      Hi @yaakov-smith_7984 ,

      Thanks for the additional details; so it sounds like the issue is that ProGet's Active Directory Integration does not support Active Directory group lookup using a "pre-Windows 2000" Group Name?

      I'm not sure if this is something we'll want to support in the Active Directory integration. That integration is sensitive and performance-critical, so trying to add-in support for group names on domain configurations from the 1990's doesn't make a lot of sense.

      For reference, here is our Active Directory Integration code:
      https://github.com/Inedo/inedox-inedocore/tree/master/InedoCore/InedoExtension/UserDirectories/ActiveDirectory

      Why Entra is using those ancient fields is beyond me, especially since all AD objects already have a unique ID. I suspect those are simply not intended to be queried or used by third-party tools. Maybe they're unstable and subject to change. I would try to get ahold of someone at Microsoft to get some answers on how to handle this.

      That said, did you try the OpenLDAP/Generic directory? That will allow you to define your own LDAP queries. I think that's what you'll need to do.

      Thanks,
      Steve

      1 Reply Last reply Reply Quote 0
      • Y Offline
        yaakov.smith_7984
        last edited by

        I don't think the issue is that ProGet's Active Directory Integration does not support Active Directory group lookup using a "pre-Windows 2000" Group Name as much as ProGet's Active Directory Integration does not support any group where the "pre-Windows 2000" Group Name differs from the Group Name, which I believe is where name and sAMAccountName differ.

        When looking up what groups a user is a member of, that system relies upon group name.
        When assigning permissions to a group, that system relies upon the pre-2K name.

        Those two systems aren't talking to eachother unless the values happen to match (which to be fair is going to be the 99% case).

        Having the link to the code now, I think the bug is here:

        https://github.com/Inedo/inedox-inedocore/blob/master/InedoCore/InedoExtension/UserDirectories/PrincipalId.cs#L51-L54

        This hardcodes sAMAccountName (i.e. "PGUSRS") as the primary source of group name, and only falls back to name if it is not present.

        But then, when checking the memberOf property of the user, that is done here against the group's full X.500-style name ("CN=proget_test_user,DC=proget,DC=test")

        https://github.com/Inedo/inedox-inedocore/blob/master/InedoCore/InedoExtension/UserDirectories/Clients/DirectoryServicesLdapClient.cs#L103-L107

        Even though the "group name property name" is overridable (https://github.com/Inedo/inedox-inedocore/blob/master/InedoCore/InedoExtension/UserDirectories/ActiveDirectory/ADUserDirectoryV4.cs#L113-L118) neither of these code paths use it, so changing that doesn't help.

        I haven't tried OpenLDAP/Generic yet, I may give that a try this afternoon.

        1 Reply Last reply Reply Quote 0
        • Y Offline
          yaakov.smith_7984
          last edited by

          I can kind of get it working under OpenLDAP/Generic directory with the "Test User Directories" tool by setting the queries to match AD and then setting LDAP Object > Group Name Property Value to distinguishedName. However, if I then try and assign any privileges to that group, it doesn't save it and there are no errors logged.

          stevedennisS 1 Reply Last reply Reply Quote 0
          • stevedennisS Offline
            stevedennis inedo-engineer @yaakov.smith_7984
            last edited by

            Hi @yaakov-smith_7984 , sounds like progress.... can you try to do a "group lookup" using the AD testing tool (page)? That's what the permissions page will do prior to saving.

            That might give you a clue to the queries that are happening.

            Y 1 Reply Last reply Reply Quote 0
            • Y Offline
              yaakov.smith_7984 @stevedennis
              last edited by

              @stevedennis
              It can't find the group on its own:
              63d86010-263d-4845-ae26-432f586a8a2c-image.png
              5836c590-b315-407e-b774-b499d5cd6af4-image.png
              But it does find it as part of all groups:
              adf548ed-9731-4795-895e-ba566e467f54-image.png
              b35a21f6-03a6-40e6-abe4-aa9318f0587d-image.png

              This whole approach seems a lot more finnicky. The AD solution seems to be clearer and more reliable, it just has that bug with the mismatching object properties.

              stevedennisS 1 Reply Last reply Reply Quote 0
              • stevedennisS Offline
                stevedennis inedo-engineer @yaakov.smith_7984
                last edited by

                @yaakov-smith_7984 good news -- we're working on a new version of the Active Directory integration that may address this problem -- want to give it a shot?

                It's in our pre-release feed:
                https://docs.inedo.com/docs/proget/administration/extensions#pre-releases

                The extension is InedoCore 3.0.5-CI.2:
                https://proget.inedo.com/feeds/PrereleaseExtensions/inedox/InedoCore/3.0.5-CI.2

                We have been running it in our environment for a while.

                This would be an ideal place to fix the issue, or at least add some kind of option to make it work better.

                1 Reply Last reply Reply Quote 0
                • Y Offline
                  yaakov.smith_7984
                  last edited by

                  a-ha!

                  v5 seems to work.

                  It has the same problems with the Test User Directories dialog, it says that my user is a member of proget_users (correct) but does not say that my user is a member of PGUSERS (incorrect).

                  However, that fades into irrelevancy - when I go assign a task/permission to the group, it saves it as proget_users@proget.test and not as PGUSERS@proget.test, so then when I log in it can follow the entire chain from proget_test_user -> memberof proget_users -> has Administer right.

                  How far along is v5? Is that prerelease build stable enough that I could give it a spin in our current production environment, or should I wait until its released as stable?

                  stevedennisS 1 Reply Last reply Reply Quote 0
                  • stevedennisS Offline
                    stevedennis inedo-engineer @yaakov.smith_7984
                    last edited by

                    @yaakov-smith_7984 great news!

                    We are planning to include it as is in the next maintenance release, later today. So it's "stable enough" we figure :)

                    1 Reply Last reply Reply Quote 0
                    • Y Offline
                      yaakov.smith_7984
                      last edited by

                      Is the v5 provider available on Linux, or only on Windows?

                      We have a new setup we're building using the container image, and AD v5 doesn't show up in Version 2025.8 (Build 17).

                      atrippA 1 Reply Last reply Reply Quote 0
                      • atrippA Offline
                        atripp inedo-engineer @yaakov.smith_7984
                        last edited by

                        Hi @yaakov-smith_7984 ,

                        It looks like we forgot to ship the updated extension in 2025.8, so it's not "included" when you install ProGet. You should be able to update under Admin > Extensions, but unless you make the extensions directory (under Admin > Advacned Configuration) mapped to the docker host it will not "survive" a container reboot.

                        We could get you a prerelease container version if you'd like to try it as included, just let us know

                        Thanks,
                        Alana

                        1 Reply Last reply Reply Quote 0

                        Hello! It looks like you're interested in this conversation, but you don't have an account yet.

                        Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

                        With your input, this post could be even better 💗

                        Register Login
                        • 1 / 1
                        • First post
                          Last post
                        Inedo Website Home • Support Home • Code of Conduct • Forums Guide • Documentation