Skip to content

[PENG-251] Apply FZ patches to goframe from bitbucket #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

jamest-pin
Copy link

@jamest-pin jamest-pin commented May 16, 2025

📝 PR Description

Todo:

Pre merge:

  • Remove repository from fork network?
  • Add github CI to run go tests? (go test -v ./...)?
  • Add security scanning??

Post merge:

  • Add tag (v0.1.0) so that Golang can pull this with go get or go mod download (specifically, from our go-proxy-core project)

Change Rollback Plan

  • not required

Change Checklist

The following must be reviewed and signed off by the creator of the PR. To sign off review and check each

🔑 Security

  • Security of any environment is not reduced by this change.
  • All existing security controls remain in place or have been replaced with equal or stronger controls.

Testing of Changes

  • Testing results have been verified as complete and covering all aspects of the change.
  • Evidence of testing attached
  • Security scans are passing
  • No package vulnerabilities introduced

🔎 Quality assurance

  • Reviewed this PR myself
  • Automated tests are required and included
  • Security controls have been reviewed
  • Changes are Blue-Green friendly

@jamest-pin jamest-pin force-pushed the PENG-251-upiproxy-fork-and-patch-goframe branch from cbe1ed4 to 0f5a4d7 Compare May 21, 2025 06:44
@jamest-pin jamest-pin requested a review from acunningham-fz May 21, 2025 06:45
@jamest-pin jamest-pin requested a review from a team May 29, 2025 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant