Skip to content
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

Issue #50 is not fixed in .Net core 1.0 #54

Closed
jyothi530 opened this issue Sep 23, 2017 · 7 comments
Closed

Issue #50 is not fixed in .Net core 1.0 #54

jyothi530 opened this issue Sep 23, 2017 · 7 comments

Comments

@jyothi530
Copy link

jyothi530 commented Sep 23, 2017

Thanks for looking into the issue #50 and fixing it but unfortunately the fix you posted is working only for .Net core 2.0 but it is not working in .Net core 1.0.
Please find the pull request #53 which contains the sample for .Net core 1.0 and also that shows it is not working.
To test in .Net core 1.0 I added a 'Name' property to ISession. This property is being set inside the command handler(CreateInventoryItemCommandHandler) when a new inventory is being created I am setting the Name property with the inventory name if it is not empty . When you look at subsequent requests the Name property value is the name of the first inventory item.
Also ServiceProvider class is not available in .Net core 1.0
Can you please look into it.

@gautema
Copy link
Owner

gautema commented Sep 25, 2017

I might get around to look at this, but since it's an issue with an old version of asp.net and not cqrslite, I'm not going to prioritize it very high.

@gautema
Copy link
Owner

gautema commented Sep 27, 2017

Fixed in next release. Is in prerelease now

@gautema gautema closed this as completed Sep 27, 2017
@artstalker
Copy link

artstalker commented Oct 13, 2017

Hi. Was it released already? I haven't found the commit.
The fix of this issue is very critical for us and we are on .Net Core 1.1 at this moment.

@gautema
Copy link
Owner

gautema commented Oct 13, 2017

It was fixed for the sample in commit 6b469d3
This isn't a change to cqrslite, but in the sample. I don't use .net core 1.1 my self, so haven't done anything to test this.

@artstalker
Copy link

artstalker commented Oct 13, 2017

Thanks for quick response!
But I still do not get it. This issue #54 was about the fact that #50 has been fix in .Net Core 2.0 only. jyothi530 asked to have a look at it. 5 days later you wrote that it's fixed. But 6b469d3 is the original fix for .Net Core 2.0. I don't see any fixes for .Net Core 1.0.

@gautema
Copy link
Owner

gautema commented Oct 14, 2017

I can understand you are confused. I think I must have been myself. I think I meant to comment on another issue when I closed this a few weeks ago, and last night I thought you were talking about 2.0. I have not done anything more with this issue. Sorry about the mix up.

@artstalker
Copy link

Ok, thanks for the clarification. Actually, we found the way how to fix it in 1.1 .Net Core. At least in the old version of CQRSLite ( we are using 0.12.9). There is DependencyResolver that injected into CQRSLite. We always create a new scope inside this DependencyResolver when resolving the services and in this way, we avoid singleton behavior for the scoped objects.
image

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

No branches or pull requests

3 participants