[ 
https://issues.apache.org/jira/browse/SOLR-18238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18138015#comment-18138015
 ] 

Eric Pugh commented on SOLR-18238:
----------------------------------

I suspect that as we get to the end of the V2 migration journey, we will take 
what we have learned and go back and make some sweeping "across our apis" 
changes.   Like idnetifying certain validations that really should be there, 
and then doing it across all of them that would benefit.    

 

Or we save this for our V3 API!  (just kidding, mostly).

> Add Stronger Validation to V2 API Parameters
> --------------------------------------------
>
>                 Key: SOLR-18238
>                 URL: https://issues.apache.org/jira/browse/SOLR-18238
>             Project: Solr
>          Issue Type: Bug
>          Components: v2 API
>    Affects Versions: 10.0
>            Reporter: Eric Pugh
>            Priority: Minor
>
> We have very inconsistent validation across our V2 apis for things like 
> pathParam and queryParams.   For example, I recently discovered that in the 
> ConfigSets API DELETE it would accept a "     " as a configset name!   
> It would be interesting to explore if we could have some validation that 
> worked across our Java v2 apis, and maybe could also be used by the Solr UI 
> as well.   
>  
> I can imagine:
>  * Investigating the Jax RS Bean validation code.
>  * Establishing much stronger patterns in our own Java API code that 
> validates using the same util methods across all APIS (strong patterns make 
> it easier for the next person to do the right thing)
>  * Something around generating validation code in the OpenAPI spec generated 
> language clients?
>  
> This would be a great ticket for someone familiar with modern web dev 
> techniques to contribute to Solr!



--
This message was sent by Atlassian Jira
(v10.3.26#10030026)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to