epugh commented on code in PR #3690:
URL: https://github.com/apache/solr/pull/3690#discussion_r2377121911


##########
solr/test-framework/src/java/org/apache/solr/BaseDistributedSearchTestCase.java:
##########
@@ -122,14 +122,15 @@ public static void initialize() {
 
   @SuppressWarnings("deprecation")

Review Comment:
   I dislike that we have chosen to deprecate 
systemSetPropertyEnableUrlAllowList, but then suppress the warning everywhere.  
 I would *fix* the problem if I know quite what to do...   @dsmiley  @tflobbe 
thoughts on taking systemSetPropertyEnableUrlAllowList and removing the 
deprecation tag?   It seems like it does exactly what it is supposed to do....



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to