-
Notifications
You must be signed in to change notification settings - Fork 10.4k
Add Ability to Tweak JWT Refresh Intervals #14909
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
Conversation
|
Is this intended to help address #13046? Or some other active issue? |
If we're going to do this it should also be done for OpenIdConnect. |
Done
Done
Partially though I'm basically trying to use the simple |
What kind of build issues? The "Build Code Check" build is asking you to update the reference assemblies. |
When I try to run I get a bunch of errors like:
|
Make sure you run restore.cmd first. |
Building of C# project is enabled and has dependencies on NodeJS projects. Building of NodeJS projects is enabled since node is detected in C:\Program Files.
Detected JDK in C:\Program Files\Java\jdk1.8.0_211 (via JAVA_HOME)
It was not possible to find any installed .NET Core SDKs
Did you mean to run .NET Core SDK commands? Install a .NET Core SDK from:
https://aka.ms/dotnet-download
C:\Users\syeh\.nuget\packages\microsoft.dotnet.arcade.sdk\1.0.0-beta.19462.4\tools\Tools.proj : error : Unable to locat
e the .NET Core SDK. Check that it is installed and that the version specified in global.json (if any) matches the inst
alled version.
C:\Users\syeh\.nuget\packages\microsoft.dotnet.arcade.sdk\1.0.0-beta.19462.4\tools\Tools.proj : error MSB4236: The SDK
'Microsoft.NET.Sdk' specified could not be found.
Build failed. |
Hmm, thanks for trying. It's working locally for me so I updated the refs and pushed to your branch. |
Thanks. Out of curiosity. Will this get backported to 3.X? |
Unlikely, this doesn't unblock any critical scenarios, it just makes existing scenarios slightly easier. |
@blowdart for triage today. |
@aspnet/asp-net-api-reviews @davidfowl @rynowak API review pls |
src/Security/Authentication/OpenIdConnect/src/OpenIdConnectOptions.cs
Outdated
Show resolved
Hide resolved
ping @aspnet/asp-net-api-reviews @davidfowl @rynowak API review pls |
src/Security/Authentication/OpenIdConnect/src/OpenIdConnectOptions.cs
Outdated
Show resolved
Hide resolved
Co-Authored-By: Stephen Halter <[email protected]>
This comment was made automatically. If there is a problem contact [email protected]. I've triaged the above build. I've created/commented on the following issue(s) |
Thanks |
Summary of the changes (Less than 80 chars)