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

New feature: IisMimeTypeMapping can update the mime type for an existing extension #263

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

richardhauer
Copy link

@richardhauer richardhauer commented Nov 30, 2016

Resolves issue #262. This PR addresses the case where there is already a defined mapping for an extension but the mimeType value is different. In this case the Set-TargetResource will update the existing item, rather than attempt to create a new one, which results in a key violation (as this element group is keyed on @fileExtension).


This change is Reviewable

…rgetResource to account for cases where extension exists but MIME type is different (and therefore should be updated)
@msftclas
Copy link

Hi @richardhauer, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@mbreakey3
Copy link
Contributor

Looks like you just need to switch your tabs to spaces.

@mbreakey3 mbreakey3 added the waiting for author response The pull request is waiting for the author to respond to comments in the pull request. label Nov 30, 2016
@richardhauer
Copy link
Author

richardhauer commented Nov 30, 2016 via email

@mbreakey3
Copy link
Contributor

You shouldn't have to... As long as your forked branch is still hooked up this PR

@tysonjhayes
Copy link

Hi @richardhauer are you till working on this? Happy to help if needed, but I'll need you to sign the cla as well.

@nickdurcholz
Copy link

We would very much like to see this PR fixed. What needs to be done in order to move it along?

@johlju
Copy link
Member

johlju commented Apr 23, 2018

@richardhauer are you still able to work on this pull request? If so I can help you review it and get it merged. We should have tests that test this scenario. If you can't work on this pull request, no worries, then I will label it as abandoned so that someone else can take over the work. 🙂

@johlju
Copy link
Member

johlju commented May 23, 2018

Labeling this PR as abandoned since it has gone 14 days or more since the last update. An abandoned PR can be continued by another contributor. The abandoned label will be removed if work on the PR is taken up again.

@johlju johlju added abandoned The pull request has been abandoned. and removed waiting for author response The pull request is waiting for the author to respond to comments in the pull request. labels May 23, 2018
@richardhauer
Copy link
Author

richardhauer commented May 23, 2018 via email

@johlju
Copy link
Member

johlju commented May 24, 2018

@richardhauer No worries. If you have time to continue working on it later on, ping me and I help you review it.

@SteveL-MSFT SteveL-MSFT added this to Abandoned in powershell/dscresources May 14, 2019
@SteveL-MSFT SteveL-MSFT removed this from Abandoned in powershell/dscresources Nov 27, 2019
@johlju johlju changed the base branch from dev to master December 30, 2019 12:20
@johlju johlju changed the base branch from master to main January 7, 2021 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
abandoned The pull request has been abandoned.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants