You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2021/03/10 23:47:22 UTC

[GitHub] [superset] robdiciuccio commented on pull request #13509: fix: added parser to parse request referer url for url_param (issue_id : 13315)

robdiciuccio commented on pull request #13509:
URL: https://github.com/apache/superset/pull/13509#issuecomment-796295860


   Thanks for opening a PR @celestialmechanic21! 
   
   Rather than relying on the Referer header, it may make more sense to pass the query params directly on the XHR request, as this would be more explicit.
   
   @villebro @dpgaspar @betodealmeida any additional context on this?


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org