Timeline for asp.net core - get User at Service Layer
Current License: CC BY-SA 4.0
19 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Jul 22, 2022 at 4:20 | comment | added | FatTiger | Injecting SessionProvider as a singleton is not thread safe, a new SessionProvider instance should be allocated for each request, so it is best to inject as Scoped | |
| Sep 1, 2018 at 15:56 | answer | added | Alex Herman | timeline score: 3 | |
| Aug 4, 2018 at 18:06 | history | edited | Alex Herman | CC BY-SA 4.0 | added 590 characters in body |
| Aug 4, 2018 at 18:02 | review | Close votes | |||
| Aug 6, 2018 at 19:08 | |||||
| Aug 4, 2018 at 18:01 | comment | added | t3chb0t | Generally you just need to post what is relevant to your question. If you think other parts might be important then why not, although Startup.csis mostly just registration. If you're not doing anything weird there, then you might skip it. BUT when you post something, please make sure it's not simplified or edited in anyway that would make it incomplete. The best you can do is just copy/paste your actual code ;-) | |
| Aug 4, 2018 at 17:58 | comment | added | Alex Herman | @t3chb0t My app consists of 2 solutions - Angular 6 and ASP.NET Core. Could you please give me more details as to which files are required for the question? Am I right in thinking adding Controller would do? Or do I also need to attach code from Startup.cs? | |
| Aug 4, 2018 at 17:54 | comment | added | t3chb0t | How about the rest? Do we see the actual code or is it edited too? ;-] | |
| Aug 4, 2018 at 17:52 | comment | added | Alex Herman | @t3chb0t Right, pardon my clumsiness with this one. Right now VocabularyService is complete. Thanks for pointing that out! | |
| Aug 4, 2018 at 17:51 | history | edited | Alex Herman | CC BY-SA 4.0 | added 109 characters in body |
| Aug 4, 2018 at 17:49 | comment | added | t3chb0t | Not really. It shouldn't look good. It shoud be real. _vocabularyHighPerformanceService = vocabularyHighPerformanceService; - this line wouldn't compile. there is no vocabularyHighPerformanceService argument. Why don't you just copy/paste what you have? | |
| Aug 4, 2018 at 17:48 | comment | added | Alex Herman | @t3chb0t Yeah sorry, my oversight. Looks better now? | |
| Aug 4, 2018 at 17:47 | history | edited | Alex Herman | CC BY-SA 4.0 | added 131 characters in body |
| Aug 4, 2018 at 17:46 | comment | added | t3chb0t | Well, this is still wrong. You're using variables that aren't even initialized :-| On Code Review we require complete and working code. Pseudocode or incomplete code is off-topic. | |
| Aug 4, 2018 at 17:43 | history | edited | Alex Herman | CC BY-SA 4.0 | added 110 characters in body |
| Aug 4, 2018 at 17:42 | comment | added | t3chb0t | What's with that: ...? It looks like you have removed parts of you code. Please don't do it. It'll backfire - it always does. Post what you have unchanged. | |
| Aug 4, 2018 at 17:41 | comment | added | Alex Herman | @t3chb0t I've got a recommendation to use codereview portal for questions like this. | |
| Aug 4, 2018 at 17:40 | comment | added | t3chb0t | from StackOverflow community - have you cross-posted it? This is Code Review ;-) | |
| Aug 4, 2018 at 17:35 | review | First posts | |||
| Aug 5, 2018 at 17:22 | |||||
| Aug 4, 2018 at 17:34 | history | asked | Alex Herman | CC BY-SA 4.0 |