Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The nuget library did not list all required libraries #46284

Closed
chucklu opened this issue Dec 21, 2020 · 7 comments
Closed

The nuget library did not list all required libraries #46284

chucklu opened this issue Dec 21, 2020 · 7 comments
Labels
area-Infrastructure-libraries untriaged New issue has not been triaged by the area owner

Comments

@chucklu
Copy link

chucklu commented Dec 21, 2020

https://www.nuget.org/packages/Microsoft.IdentityModel.JsonWebTokens/6.8.0

.NETFramework 4.5
    Microsoft.IdentityModel.Tokens (>= 6.8.0)
.NETFramework 4.6.1
    Microsoft.IdentityModel.Tokens (>= 6.8.0)
.NETStandard 2.0
    Microsoft.IdentityModel.Tokens (>= 6.8.0)

packages\Microsoft.IdentityModel.JsonWebTokens.6.8.0\lib\net45\Microsoft.IdentityModel.JsonWebTokens.dll
packages\Microsoft.IdentityModel.JsonWebTokens.6.8.0\lib\net461\Microsoft.IdentityModel.JsonWebTokens.dll
packages\Microsoft.IdentityModel.JsonWebTokens.6.8.0\lib\netstandard2.0\Microsoft.IdentityModel.JsonWebTokens.dll

The above three version net45, net461 and .net standard 2.0 verison of Microsoft.IdentityModel.JsonWebTokens both required another nuget library Microsoft.IdentityModel.Logging when you check the references of Microsoft.IdentityModel.JsonWebTokens through dnSpy.
Microsoft.IdentityModel.Logging should be list as dependency of Microsoft.IdentityModel.JsonWebTokens. (Although Microsoft.IdentityModel.Tokens requires Microsoft.IdentityModel.Logging)

@joperezr @iSazonov Maybe similar problem like #46182? something wrong with the tool used in Microsoft

@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added area-DependencyModel untriaged New issue has not been triaged by the area owner labels Dec 21, 2020
@ghost
Copy link

ghost commented Dec 21, 2020

Tagging subscribers to this area: @eerhardt
See info in area-owners.md if you want to be subscribed.

Issue Details

https://www.nuget.org/packages/Microsoft.IdentityModel.JsonWebTokens/6.8.0

.NETFramework 4.5
    Microsoft.IdentityModel.Tokens (>= 6.8.0)
.NETFramework 4.6.1
    Microsoft.IdentityModel.Tokens (>= 6.8.0)
.NETStandard 2.0
    Microsoft.IdentityModel.Tokens (>= 6.8.0)

packages\Microsoft.IdentityModel.JsonWebTokens.6.8.0\lib\net45\Microsoft.IdentityModel.JsonWebTokens.dll
packages\Microsoft.IdentityModel.JsonWebTokens.6.8.0\lib\net461\Microsoft.IdentityModel.JsonWebTokens.dll
packages\Microsoft.IdentityModel.JsonWebTokens.6.8.0\lib\netstandard2.0\Microsoft.IdentityModel.JsonWebTokens.dll

The above three version net45, net461 and .net standard 2.0 verison of Microsoft.IdentityModel.JsonWebTokens both required another nuget library Microsoft.IdentityModel.Logging when you check the references of Microsoft.IdentityModel.JsonWebTokens through dnSpy.
Microsoft.IdentityModel.Logging should be list as dependency of Microsoft.IdentityModel.JsonWebTokens. (Although Microsoft.IdentityModel.Tokens requires Microsoft.IdentityModel.Logging)

@joperezr @iSazonov Maybe similar problem like #46182? something wrong with the tool used in Microsoft

Author: chucklu
Assignees: -
Labels:

area-DependencyModel, untriaged

Milestone: -

@chucklu chucklu changed the title The nuget library did not list all required library The nuget library did not list all required libraries Dec 21, 2020
@iSazonov
Copy link
Contributor

Doesn't nuget site always display only the top level of dependencies?

@eerhardt
Copy link
Member

The Microsoft.IdentityModel.JsonWebTokens isn't built from the dotnet/runtime repo. According to nuget.org, the source repository for this package is: https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet.

@ghost
Copy link

ghost commented Dec 21, 2020

Tagging subscribers to this area: @safern, @ViktorHofer
See info in area-owners.md if you want to be subscribed.

Issue Details

https://www.nuget.org/packages/Microsoft.IdentityModel.JsonWebTokens/6.8.0

.NETFramework 4.5
    Microsoft.IdentityModel.Tokens (>= 6.8.0)
.NETFramework 4.6.1
    Microsoft.IdentityModel.Tokens (>= 6.8.0)
.NETStandard 2.0
    Microsoft.IdentityModel.Tokens (>= 6.8.0)

packages\Microsoft.IdentityModel.JsonWebTokens.6.8.0\lib\net45\Microsoft.IdentityModel.JsonWebTokens.dll
packages\Microsoft.IdentityModel.JsonWebTokens.6.8.0\lib\net461\Microsoft.IdentityModel.JsonWebTokens.dll
packages\Microsoft.IdentityModel.JsonWebTokens.6.8.0\lib\netstandard2.0\Microsoft.IdentityModel.JsonWebTokens.dll

The above three version net45, net461 and .net standard 2.0 verison of Microsoft.IdentityModel.JsonWebTokens both required another nuget library Microsoft.IdentityModel.Logging when you check the references of Microsoft.IdentityModel.JsonWebTokens through dnSpy.
Microsoft.IdentityModel.Logging should be list as dependency of Microsoft.IdentityModel.JsonWebTokens. (Although Microsoft.IdentityModel.Tokens requires Microsoft.IdentityModel.Logging)

@joperezr @iSazonov Maybe similar problem like #46182? something wrong with the tool used in Microsoft

Author: chucklu
Assignees: -
Labels:

area-Infrastructure-libraries, untriaged

Milestone: -

@eerhardt
Copy link
Member

Moving to area-Infrastructure-libraries to match #46182

@ViktorHofer
Copy link
Member

As Eric said, this issue doesn't belong to dotnet/runtime and should be moved.

@chucklu
Copy link
Author

chucklu commented Dec 22, 2020

@iSazonov both Microsoft.IdentityModel.Logging and Microsoft.IdentityModel.Tokens are top level dependencies. Anyway I will file a new issue in correct repository https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet .

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Infrastructure-libraries untriaged New issue has not been triaged by the area owner
Projects
None yet
Development

No branches or pull requests

5 participants