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

Change object IWebHostEnvironment for new IHostEnvironment of net core #111

Merged
merged 20 commits into from
Jun 18, 2020

Conversation

dadjh85
Copy link
Contributor

@dadjh85 dadjh85 commented Jun 18, 2020

Hi @Burgyn,

I have detected something interesting in the code of the AddOcelotWithSwaggerSupport extension, and it is that you are using the IWebHostEnvironment object, this object changed in the .NET Core 3.1 version for IHostEnvironment, although you can still use the old object it would be quite interesting to use the new object for future versions of .NET Core

David Jiménez and others added 20 commits May 18, 2020 23:48
* Revert default value of serverOverride
* Added .RemoveSlashFromEnd(); in serverOverride
# Conflicts:
#	README.md
#	demo/ApiGateway/Startup.cs
#	src/MMLib.SwaggerForOcelot/DependencyInjection/ServiceCollectionExtensions.cs
#	src/MMLib.SwaggerForOcelot/Middleware/SwaggerForOcelotMiddleware.cs
#	src/MMLib.SwaggerForOcelot/ReRouteOptionsExtensions.cs
#	src/MMLib.SwaggerForOcelot/ServiceDiscovery/SwaggerServiceDiscoveryProvider.cs
#	src/MMLib.SwaggerForOcelot/Transformation/ISwaggerJsonTransformer.cs
#	src/MMLib.SwaggerForOcelot/Transformation/SwaggerJsonTransformer.cs
#	tests/MMLib.SwaggerForOcelot.Tests/ServiceDiscovery/SwaggerServiceDiscoveryProviderShould.cs
#	tests/MMLib.SwaggerForOcelot.Tests/SwaggerForOcelotMiddlewareShould.cs
# Conflicts:
#	demo/ApiGateway/Program.cs
#	src/MMLib.SwaggerForOcelot/DependencyInjection/ConfigurationBuilderExtensions.cs
@Burgyn Burgyn merged commit b5e4085 into Burgyn:master Jun 18, 2020
@dadjh85
Copy link
Contributor Author

dadjh85 commented Jun 18, 2020

Okay, fine, you're gonna republish the nuget now?, for waiting for me to update my code where I use this nuget

@Burgyn
Copy link
Owner

Burgyn commented Jun 18, 2020

Yes, I republish new version 2.5.0 in a few minutes.

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