185 more restrictive api for string arguments

[forkimenjeckayang]

• Modified some REST APIs in datasafe-rest-impl to enhance security with input / parameter validation

[max402]

It looks like original issue was fixed long time ago. URI was replaced with url-safe Uri wrapper class. The TODO in BasePrivateResource can be removed.

[forkimenjeckayang]

Noted Sir @max402 , i noticed too that's why there was little or nothing to do with the rest apis in datasafe-rest-impl. And by removing the TODO in BasePrivateResource you mean everything from line 30 and below or just the 1 arg constructor ?

[max402]

@forkimenjeckayang No, I mean remove only the TODO comment line

[forkimenjeckayang]

Oh okay , just from doing that.

[max402]

@forkimenjeckayang please rebase you branch on top of current develop

[forkimenjeckayang]

OK @max402 , i have done that

[forkimenjeckayang]

Hello @max402 Should i go ahead and merge then close or you will do it