我有一个MVC控制器,旨在从HTTP标头中读取用户的信息。标头包含从单点登录(SiteMinder)返回的用户信息。 SiteMinder将用户重定向到公司登录页面,并返回我的应用程序以及cookie和HTTP标头。 HTTP标头包含用户的信息。我的控制器应该获取这些信息并将其显示在导航栏中。
这是控制器代码:
[HttpGet]
public async Task<IActionResult> GetUser()
{
var person = Request.Headers["HTTP_JHED_UID"].ToString();
if (repository.GetJhedUser(person) != null)
{
var user = await repository.GetUser(person);
var userDto = mapper.Map<User, UserForDisplayDto>(user);
return Ok(userDto);
}
else
{
return null;
}
}
这是我对应的存储库:
public string GetJhedUser(string value)
{
return context.Users.Where(x => x.JHED_ID == value).ToString();
}
public async Task<User> GetUser(string id, bool includeRelated = true)
{
if(!includeRelated)
return await context.Users.FindAsync(id);
return await context.Users
.SingleOrDefaultAsync(s => s.JHED_ID == id);
}
我收到500服务器错误。我确信标头有这些值。
编辑:这是调试的图像。即使我尝试使用“日期”,价值也没有。请注意,我将我的代码重构为@ Nkosi的代码
如何从标题中获取这些用户的信息(在本例中只是用户名),以便我可以在我的HTML模板中显示?
答案 0 :(得分:1)
不要从控制器操作返回null
。
在尝试访问之前检查标头是否确实存在。
在防守方面重新构建要编码的动作。
[HttpGet]
public async Task<IActionResult> GetUser() {
//was the header provided?
var headerValue = Request.Headers["HTTP_JHED_UID"];
if(headerValue.Any() == false) return BadRequest(); //401
//does the person exist?
var person = headerValue.ToString();
if(repository.GetJhedUser(person) == null) return NotFound(); //404
var user = await repository.GetUser(person);
var userDto = mapper.Map<User, UserForDisplayDto>(user);
return Ok(userDto); //200
}
GetJhedUser
的存储库代码也存在问题,因为在linq表达式上调用string并没有按照您的想法执行操作。
我会进一步建议将GetJhedUser
重构为实际检查的内容
public bool JhedUserExists(string value) {
return context.Users.FirstOrDefault(x => x.JHED_ID == value) != null;
}
这导致行动看起来像这样
[HttpGet]
public async Task<IActionResult> GetUser() {
//was the header provided?
var headerValue = Request.Headers["HTTP_JHED_UID"];
if(headerValue.Any() == false) return BadRequest(); //401
//does the person exist?
var person = headerValue.ToString();
if(repository.JhedUserExists(person) == false) return NotFound(); //404
var user = await repository.GetUser(person);
var userDto = mapper.Map<User, UserForDisplayDto>(user);
return Ok(userDto); //200
}