Why doesn't OwinEnvironment implement IEnumerable.GetEnumerator?

[kchanlee]

In the project Microsoft.AspNetCore.Owin, OwinFeatureCollection implements IEnumerable.GetEnumerator() which looks good.

        IEnumerator IEnumerable.GetEnumerator()
        {
            return GetEnumerator();
        }

        public IEnumerator<KeyValuePair<Type, object>> GetEnumerator()
        {
            ...
        }

Why doesn't OwinEnvironment implements IEnumerable.GetEnumerator() same as OwinFeatureCollection? Now (dev branch, Commit 2cafa43) it just

        IEnumerator IEnumerable.GetEnumerator()
        {
            throw new NotImplementedException();
        }

[JunTaoLuo]

Most likely an oversight. However, why are you using the IEnumerable.GetEnumerator specifically instead of the generic IEnumerable<KeyValuePair<string, object>>.GetEnumerator

[davidfowl]

@kchanlee would you like to send a PR?

[kchanlee]

@JunTaoLuo Actually I don't have a real use case. I was reading and learning from the code and found it.

[kchanlee]

@davidfowl Sure, I am willing to contribute. Here it is.