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

fix issue with https binding .net core 3.0 support #1283 #1286

Merged
merged 1 commit into from
Feb 15, 2021

Conversation

luber
Copy link
Contributor

@luber luber commented Feb 14, 2021

commenting out Kestrel listenOptions.NoDelay b/c:

  1. the default value = true
  2. it was removed in .net core 3.0 and this can be configured in SocketTransportOptions

@CLAassistant
Copy link

CLAassistant commented Feb 14, 2021

CLA assistant check
All committers have signed the CLA.

@mregen mregen merged commit 61eede8 into OPCFoundation:master Feb 15, 2021
@matteotrapani
Copy link

Hi @luber

Thank you for fixing this!
Meanwhile how can I overcome this issue?
Do you know when this fix will be released?

Thanks!

@luber
Copy link
Contributor Author

luber commented Feb 18, 2021

Hi @matteotrapani
I believe to overcome the issue you can host the app using .net core 2.2 instead of 3.0.
Unfortunately, I do not know about any release plans. It would be great if someone, who is responsible for this, creates a milestone for the vNext version with some target date so everybody could use it during planning...

@matteotrapani
Copy link

@mregen or @AlinMoldovean could you help us in some way?

@mregen
Copy link
Contributor

mregen commented Feb 18, 2021

@matteotrapani could you start using the preview nuget packages from the preview feed?
I think we will do a last 365 release before we move to the model compiler updates with the V1.04.10 update, as the 366 may require more testing.

@matteotrapani
Copy link

Hi @mregen ,

Thank you for your answer.
I didn't know the existence of a specific feed for preview packages, I just looked into the Nuget feed.
Everything seem to work fine now.

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.

None yet

5 participants