[GitHub] [trafficserver] maskit opened a new pull request, #9098: Add URLImpl::get_scheme

[GitBox <gi...@apache.org>]

maskit opened a new pull request, #9098:
URL: https://github.com/apache/trafficserver/pull/9098

   `URLImpl::get_scheme` doesn't exist somehow where there are accessors for other parts, and `URL` has to touch member variables in `URLImpl` directly because of that.
   
   This add `URLImpl::get_scheme` and move the current implantation in `URL` into `URLImpl`. There should be no logic/behavior change.


-- 
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: github-unsubscribe@trafficserver.apache.org

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

[GitHub] [trafficserver] maskit merged pull request #9098: Add URLImpl::get_scheme

[GitBox <gi...@apache.org>]

maskit merged PR #9098:
URL: https://github.com/apache/trafficserver/pull/9098


-- 
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: github-unsubscribe@trafficserver.apache.org

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

[GitHub] [trafficserver] zwoop commented on pull request #9098: Add URLImpl::get_scheme

[GitBox <gi...@apache.org>]

zwoop commented on PR #9098:
URL: https://github.com/apache/trafficserver/pull/9098#issuecomment-1353622739

   Cherry-picked to v9.2.x


-- 
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: github-unsubscribe@trafficserver.apache.org

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