Skip to content
This repository was archived by the owner on Sep 21, 2018. It is now read-only.

Windows auth templates don't need the self-host option #537

Closed
Tratcher opened this issue May 3, 2016 · 7 comments
Closed

Windows auth templates don't need the self-host option #537

Tratcher opened this issue May 3, 2016 · 7 comments

Comments

@Tratcher
Copy link
Member

Tratcher commented May 3, 2016

The combination of Windows Auth and self-host launch settings does not currently make sense because Kestrel does not support windows auth. Consider not providing self-host launch settings by default for windows auth scenarios.

This may someday make sense when we add WebListener.

@rustd
Copy link
Contributor

rustd commented May 4, 2016

@Tratcher In this case launch settings would be a no-op then correct?

@Tratcher
Copy link
Member Author

Tratcher commented May 4, 2016

No, I'm suggesting not having the self-host profile at all for these scenarios.

@rustd
Copy link
Contributor

rustd commented May 4, 2016

yes i get that, but wanted to check if having them there breaks anything?

@Tratcher
Copy link
Member Author

Tratcher commented May 4, 2016

For WebApp you just get a strange Hello, ! in the corner where it tries to display the logged in user.

For WebApi it throws:

System.InvalidOperationException: No authentication handler is configured to handle the scheme: Automatic
   at Microsoft.AspNetCore.Http.Authentication.Internal.DefaultAuthenticationManager.<ChallengeAsync>d__11.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.ChallengeResult.<ExecuteResultAsync>d__14.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.Internal.FilterActionInvoker.<InvokeResultAsync>d__44.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.Internal.FilterActionInvoker.<InvokeAsync>d__32.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.Internal.MvcRouteHandler.<InvokeActionAsync>d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Builder.RouterMiddleware.<Invoke>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Hosting.Internal.RequestServicesContainerMiddleware.<Invoke>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Server.Kestrel.Http.Frame`1.<RequestProcessingAsync>d__2.MoveNext()

@rustd rustd added this to the 1.0.0-rc2 milestone May 4, 2016
@rustd
Copy link
Contributor

rustd commented May 4, 2016

ah. we need to fix this. @phenning assigning to you for RC2

@phenning
Copy link
Contributor

phenning commented May 4, 2016

TFS 218918

@phenning
Copy link
Contributor

phenning commented May 4, 2016

ffebd22

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants