Michae,

The code in the GitHub repo is targeting v1.1 – SDK 1.0.0 Preview 2.1-003177, so it is the latest code.

The example doesn’t have any other parameters it is binding to, so it doesn’t need to check for PropertyName. In your case I would just check if context.Metadata.PropertyName != null.

For your second comment, the IModelBinderProvider will get called for everything that is bound – in your case “id”, “item”, and all of item’s properties. Your provider gets to choose which of those it wants to handle the binding for, and for everything else it will return null. In the example it is only handling the binding for properties of the model with an IScrubberAttribute. Everything else is ignored.

Hope that helps.

Dan

That being the case, I really can’t determine how I would get a single IModelBinder to return since there could be multiple attributes in the Model that need to be scrubbed.

[HttpPut(“{id}”)]
public IActionResult Update(string id, [FromBody] Test item)

and in the ScrubbingModelBinderProvider.GetBinder, I get called twice:
1. with a string, the id form the Update method
2. with a Test instance from the Update method

In the GetBinder, the call for the id crashes because the PropertyName is null, the call for the Test instance does nothing because it is a complex type and the GetBinder method ignores complex types.

Thanks Ryan. Yes, the code definitely is not complete, but as-is should work. I just installed .Net Core 1.0.1 and the tooling on a machine without any .Net Core components, and the project works. What kind of error are you getting on the line in ScrubbingModelBinderProvider? Are you on version 1.1?

As for changing it to CurrencyScrubberAttribute, the binder provider is written to work with any attribute that implements IScrubberAttribute, so it should be typeof(IScrubberAttribute).

On the ScrubbingModelBinderProvider, line 14:
var attribute = propInfo.GetCustomAttributes(typeof(IScrubberAttribute), false).FirstOrDefault();

This throws an error. It should be CurrencyScrubberAttribute instead of IScrubberAttribute.

You may also want to throw in some error handling to make sure propName and propInfo are not null before you get to attribute. It was throwing errors for me before I put that in.

Your post helped me to resolve an error and understand the model binding behaviors.