GWTP
GWTP copied to clipboard
Rest Dispatch support of HTTP Digest / Basic authentication
GWT RequestBuilder already support this kind of authentication, we just need to set the attributes user and password on the RequestBuilder before calling the method send().
One thing to not forget is that not all services methods requires HTTP authentication, this mean that this new feature is an Action related feature, I was thinking to create a Sub-type of Action with this tow new additional required fields.
I'm not sure that we have to provide 'out-of-the-box' these mechanisms in Rest Dispatch. Basic authentication is not the only authentication means for REST service and it is far to be the recommended one. But I agree that we should give a way to the developers to implement it if they want to use it.
Implementing it would mean that we recommend to use it... Something that I disagree.
I think this can be easily done with the extension points available in gwtp 1.1 I suggest though that we look into the most common authentication methods and make sure they are "easy" to implement by the developer through the extension points.
Following my comment here, it will be easy to add global parameters to all requests, credentials included.
Adding direct support for the built-in Basic-Authentication through setPassword()
/ setUser()
/ setIncludeCredentials()
may just add complexity, although may also help the developer in some cases.
I still don't know if this is something I want to implement...
Let's open an issue for this, in the meanwhile, adding the ability for the user to implement authentication scheme easily MUST be a design goal. Then adding built-in support can come following usage from the community
@Chris-V do we have a concrete feature to add or should I close that issue?