Skip to content

Add admin page to act as a demo of a spa#102

Open
stimms wants to merge 7 commits intomasterfrom
ftr/spa
Open

Add admin page to act as a demo of a spa#102
stimms wants to merge 7 commits intomasterfrom
ftr/spa

Conversation

@stimms
Copy link
Contributor

@stimms stimms commented Dec 11, 2016

No description provided.


[HttpPut]
[Route("Unlock/{userId}")]
public async Task Unlock(Guid userId)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ValidateAntiForgeryToken?

@dpaquette
Copy link
Contributor

This looks good. A couple of questions?

  • Should the js files be checked in or should we only check in the tsx file?
  • Is there a potential security risk with not doing the usual anti-forgery token checks on the PUT action methods?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants