Scott Beddall

I will tackle this at roughly the same time as I tackle #972. Thanks for the heads up Jeremy!

@navba I really appreciate the effort in adding these two missing functions. Unfortunately, this PR makes changes in an autogenerated file. The very next time that the pipeline that generates...

@navba I really appreciate the effort in adding these two missing functions. Unfortunately, this PR makes changes in an autogenerated file. The very next time that the pipeline that generates...

Something went pretty wrong when force pushing the branch here. It looks like the changes from my pygithub update were individually pulled over and overrwrote what this PR already was?...

❤️ your commits on this Harsha. Thanks for the rewrite 👍

> Looks good! Thanks Harsha for the follow up. > > I did have one broader follow-up question (but happy to merge as-is) -- now that we're enforcing that the...

We haven't seen this issue recently. [DataSource](https://dataexplorer.azure.com/clusters/azsdkengsys.westus2/databases/Pipelines) ```kusto BuildLogLine | where Timestamp > ago(60d) | where Message has "ValueError: underlying buffer has been detached" | count -> 0 ```

> Should we also remove the `requests` imports in `proxy_startup.py` and `proxy_testcase.py`? **Walter White Gif**

> Hey @scbedd. Ran into some issues with `requests` not being available in certain pipeline environments ([example](https://dev.azure.com/azure-sdk/public/_build/results?buildId=2037071&view=logs&j=4fa73ede-de59-50b0-851d-db96582ea108&t=271dca1a-6c62-5d1b-458d-2dd10be9b8db&l=10994)), and that led me here, which should resolve that issue. However, after running...

/azp run python - core - ci