PSResourceGet icon indicating copy to clipboard operation
PSResourceGet copied to clipboard

`Get-PSResource` does not search in user context environment variable `PSModulePath`

Open o-l-a-v opened this issue 2 years ago • 1 comments

Prerequisites

  • [X] Write a descriptive title.
  • [X] Make sure you are able to repro it on the latest released version
  • [X] Search the existing issues.

Steps to reproduce

PackageManagement, PowerShellGetv2 and PowerShellGet 3.0.18-beta (and prior versions) does not search in directorories specified in user context environmental variable PSModulePath when searching for modules. But at least v3 has a -Path parameter.

Reproduce:

  • Install Az.Accounts to %LOCALAPPDATA%\Microsoft\PowerShell\Modules by using Save-PSResource.
  • Make sure said module is not installed any other places specified in $env:PSModulePath.
    • Or rather any other default locations for PowerShell modules I guess.
  • Make sure %LOCALAPPDATA%\Microsoft\PowerShell\Modules is specified in HKCU:\Environment\PSModulePath, and that PowerShell loaded it into $env:PSModulePath at start. Should also be seen in [System.Environment]::GetEnvironmentVariable('PSModulePath','User').
  • Search for installed modules with PowerShellGet\Get-PSResource -Name 'Az.Accounts'. Result: Not found.
  • Add -Scope 'CurrentUser: Nope, still not found.
  • Add -Path ('{0}\Microsoft\PowerShell\Modules' -f $env:LOCALAPPDATA): Found.

Microsoft.PowerShell.Core\Import-Module and Get-Module works as expected.

Expected behavior

PowerShellGet\Get-PSResource should search in all PSModulePath paths, also those specified in user context environmental variable.

Actual behavior

It does not.

Error details

No response

Environment data

PowerShellGet v3.0.18
Windows PowerShell 5.1 x64 on Windows 10 22H2

Visuals

No response

o-l-a-v avatar Jan 12 '23 22:01 o-l-a-v

Here's the responsible code as far as I can see:

https://github.com/PowerShell/PSResourceGet/blob/b28fba0fecd6736bad1a40640b9814b2bbf27cf5/src/code/Utils.cs#L993-L1017

Seems doable to add PSModulePath here?

if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows) && scope is not null) {
    Environment.GetEnvironmentVariable(
        "PSModulePath",
        scope.Value is ScopeType.CurrentUser ? EnvironmentVariableTarget.User : EnvironmentVariableTarget.Machine
    );
}

I understand it's more complex than that. One must have tests etc.

Is this one up for grabs, or do you not want environment variable PSModulePath to have any say?


Edit: Here's a PowerShellers attempt. 😁

Click to view
if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows) && scope is not null) {
    string PSModulePath = Environment.GetEnvironmentVariable(
        "PSModulePath",
        scope.Value is ScopeType.CurrentUser ? EnvironmentVariableTarget.User : EnvironmentVariableTarget.Machine
    );
    if (string.IsNullOrEmpty(PSModulePath)) {
        psCmdlet.WriteVerbose("Found no value for PSModulePath given the scope.");
    }
    else {
        foreach (string element in PSModulePath.Split(Path.PathSeparator)) {
            if (Directory.Exists(element)) {
                if (
                    string.Equals(
                        element.Split(Path.DirectorySeparatorChar).Last(),
                        "Modules",
                        StringComparison.OrdinalIgnoreCase
                    )
                ) {
                    psCmdlet.WriteVerbose(
                        string.Format(
                            "'{0}' exists, adding it to 'resourcePaths'.",
                            element
                        )
                    );
                    resourcePaths.Add(element);
                    resourcePaths.Add(Path.Combine(Directory.GetParent(element).FullName, "Scripts"));
                }
                else {
                    psCmdlet.WriteVerbose(
                        string.Format(
                            "'{0}' exists but does not match expected pattern, not adding it to 'resourcePaths'.",
                            element
                        )
                    );
                }
            }
            else {
                psCmdlet.WriteVerbose(
                    string.Format(
                        "'{0}' does not exist, not adding it to 'resourcePaths'.",
                        element
                    )
                );
            }
        }
    }
}

o-l-a-v avatar Apr 01 '24 21:04 o-l-a-v