6 ms·
I must be too tired because I cannot figure out what happened in that pull request. The PR/source branch was over a year old with none of the commits adding up
by vultour 1mo ago
I must be too tired because I cannot figure out what happened in that pull request. The PR/source branch was over a year old with none of the commits adding up to the full diff. There is [1], which introduced the vulnerability but didn't remove the environment variables above, then master is merged into it via [2] (but still doesn't show the variables being removed), yet in the full PR diff they're gone.
In any case, I'm pretty sure you misattributed the vulnerability to Copilot because the PR got squash-merged and _all_ of the changes were then attributed to every contributor in that PR, despite Copilot only appearing on one of the commits.
[1] https://github.com/snowflakedb/snowflake-connector-net/commit/094038e59d112906f1790acf39999045fc0df243 https://github.com/snowflakedb/snowflake-connector-net/commi...
[2] https://github.com/snowflakedb/snowflake-connector-net/commit/51dc7381e64c60033ec10c52939ff4ae1d3d83ab https://github.com/snowflakedb/snowflake-connector-net/commi...
- croemer 1mo agoYes, it's misattributed, a human introduced it: https://github.com/snowflakedb/snowflake-connector-net/pull/1218/changes/094038e59d112906f1790acf39999045fc0df243 https://github.com/snowflakedb/snowflake-connector-net/pull/...
- galnagli 1mo agoThanks @vultour and croemer for your proactiveness -- you are correct,I updated the blog to clarify that Copilot was a co-author that checked the merged PR and code change, and identified it as all-clear without noticing the critical vulnerabilities, it's unclear whether the code-change was AI-Assisted