-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Fix Single Page Application
example in README.md
file
#678
Merged
coreydaley
merged 10 commits into
gorilla:main
from
amustaque97:fix/update-readme-serve-spa
Aug 16, 2023
Merged
Fix Single Page Application
example in README.md
file
#678
coreydaley
merged 10 commits into
gorilla:main
from
amustaque97:fix/update-readme-serve-spa
Aug 16, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@elithrar can you please review the PR? Looking forward to hearing from you. |
elithrar
suggested changes
Jun 3, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't really mergeable.
- the example has unreachable code
- we add a number of dependencies to our tests, which don't test any code changes at all. At best, these should be in a self-contained
_example.go
file, separate from our tests. We aren't testing mux itself here.
amustaque97
force-pushed
the
fix/update-readme-serve-spa
branch
from
June 5, 2022 19:10
62b49e9
to
d308dd3
Compare
ianling
reviewed
Aug 9, 2022
Codecov Report
@@ Coverage Diff @@
## main #678 +/- ##
=======================================
Coverage 78.44% 78.44%
=======================================
Files 5 5
Lines 877 877
=======================================
Hits 688 688
Misses 135 135
Partials 54 54 |
coreydaley
approved these changes
Aug 16, 2023
oguzhand95
referenced
this pull request
in cerbos/cerbos
Nov 6, 2023
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) | require | minor | `v1.46.6` -> `v1.47.3` | | [github.com/bufbuild/protovalidate-go](https://github.com/bufbuild/protovalidate-go) | require | minor | `v0.3.1` -> `v0.4.0` | | [github.com/cerbos/cerbos/api/genpb](https://github.com/cerbos/cerbos) | require | digest | `f134903` -> `761a3dc` | | [github.com/cerbos/cloud-api](https://github.com/cerbos/cloud-api) | require | patch | `v0.1.8` -> `v0.1.9` | | [github.com/go-logr/zapr](https://github.com/go-logr/zapr) | require | minor | `v1.2.4` -> `v1.3.0` | | [github.com/gorilla/mux](https://github.com/gorilla/mux) | require | patch | `v1.8.0` -> `v1.8.1` | | [github.com/jackc/pgx/v5](https://github.com/jackc/pgx) | require | minor | `v5.4.3` -> `v5.5.0` | | [github.com/lestrrat-go/jwx/v2](https://github.com/lestrrat-go/jwx) | require | patch | `v2.0.15` -> `v2.0.16` | | [github.com/pterm/pterm](https://github.com/pterm/pterm) | require | patch | `v0.12.69` -> `v0.12.70` | | [github.com/rivo/tview](https://github.com/rivo/tview) | require | digest | `8b7bcf9` -> `1b91b81` | | [github.com/twmb/franz-go](https://github.com/twmb/franz-go) | require | patch | `v1.15.1` -> `v1.15.2` | | [github.com/vektra/mockery/v2](https://github.com/vektra/mockery) | require | patch | `v2.36.0` -> `v2.36.1` | | golang.org/x/sync | require | minor | `v0.4.0` -> `v0.5.0` | | [google.golang.org/genproto/googleapis/api](https://github.com/googleapis/go-genproto) | require | digest | `49dd2c1` -> `d783a09` | | [modernc.org/sqlite](https://gitlab.com/cznic/sqlite) | require | minor | `v1.26.0` -> `v1.27.0` | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>aws/aws-sdk-go (github.com/aws/aws-sdk-go)</summary> ### [`v1.47.3`](https://github.com/aws/aws-sdk-go/blob/HEAD/CHANGELOG.md#Release-v1473-2023-11-03) [Compare Source](https://github.com/aws/aws-sdk-go/compare/v1.47.2...v1.47.3) \=== ##### Service Client Updates - `service/config`: Updates service API - `service/connect`: Updates service API and documentation - `service/iotwireless`: Updates service API and documentation - `service/launch-wizard`: Adds new service ### [`v1.47.2`](https://github.com/aws/aws-sdk-go/blob/HEAD/CHANGELOG.md#Release-v1472-2023-11-02) [Compare Source](https://github.com/aws/aws-sdk-go/compare/v1.47.1...v1.47.2) \=== ##### Service Client Updates - `service/apprunner`: Updates service API and documentation - `service/connect`: Updates service documentation - `service/gamelift`: Updates service API and documentation - Amazon GameLift adds support for shared credentials, which allows applications that are deployed on managed EC2 fleets to interact with other AWS resources. - `service/glue`: Updates service API and documentation - This release introduces Google BigQuery Source and Target in AWS Glue CodeGenConfigurationNode. - `service/network-firewall`: Updates service API and documentation - `service/quicksight`: Updates service API and documentation - Got confirmed from qmeixua@ about custom week features, and tested locally with aws cli and java sdk that the subtypes are showing up. ##### SDK Enhancements - `aws/ec2metadata`: Added environment and shared config support for disabling IMDSv1 fallback. - Use env `AWS_EC2_METADATA_V1_DISABLED` or shared config `ec2_metadata_v1_disabled` accordingly. ### [`v1.47.1`](https://github.com/aws/aws-sdk-go/blob/HEAD/CHANGELOG.md#Release-v1471-2023-11-01) [Compare Source](https://github.com/aws/aws-sdk-go/compare/v1.47.0...v1.47.1) \=== ##### Service Client Updates - `service/connect`: Updates service API, documentation, and paginators - `service/globalaccelerator`: Updates service API, documentation, and paginators - `service/rds`: Updates service API, documentation, waiters, paginators, and examples - This release adds support for customized networking resources to Amazon RDS Custom. - `service/redshift`: Updates service API and documentation - Added support for Multi-AZ deployments for Provisioned RA3 clusters that provide 99.99% SLA availability. - `service/sagemaker`: Updates service API and documentation - Support for batch transform input in Model dashboard ### [`v1.47.0`](https://github.com/aws/aws-sdk-go/blob/HEAD/CHANGELOG.md#Release-v1470-2023-10-31) [Compare Source](https://github.com/aws/aws-sdk-go/compare/v1.46.7...v1.47.0) \=== ##### Service Client Updates - `service/amplify`: Updates service API, documentation, and paginators - `service/application-insights`: Updates service API and documentation - `service/ec2`: Updates service API, documentation, and paginators - Capacity Blocks for ML are a new EC2 purchasing option for reserving GPU instances on a future date to support short duration machine learning (ML) workloads. Capacity Blocks automatically place instances close together inside Amazon EC2 UltraClusters for low-latency, high-throughput networking. - `service/m2`: Updates service API and documentation - `service/neptunedata`: Updates service API and documentation - `service/translate`: Updates service API and documentation ##### SDK Features - `aws`: Bump minimum go version to 1.19. - See https://aws.amazon.com/blogs/developer/aws-sdk-for-go-aligns-with-go-release-policy-on-supported-runtimes/. ### [`v1.46.7`](https://github.com/aws/aws-sdk-go/blob/HEAD/CHANGELOG.md#Release-v1467-2023-10-30) [Compare Source](https://github.com/aws/aws-sdk-go/compare/v1.46.6...v1.46.7) \=== ##### Service Client Updates - `service/connect`: Updates service API and documentation - `service/dataexchange`: Updates service API and documentation - `service/datasync`: Updates service API and documentation - `service/finspace`: Updates service API and documentation - `service/mediapackagev2`: Updates service API and documentation - `service/rds`: Updates service API, documentation, waiters, paginators, and examples - This release launches the CreateIntegration, DeleteIntegration, and DescribeIntegrations APIs to manage zero-ETL Integrations. - `service/redshift-serverless`: Updates service API, documentation, and paginators - `service/resiliencehub`: Updates service API and documentation - `service/s3outposts`: Updates service API and documentation - `service/wisdom`: Updates service documentation </details> <details> <summary>bufbuild/protovalidate-go (github.com/bufbuild/protovalidate-go)</summary> ### [`v0.4.0`](https://github.com/bufbuild/protovalidate-go/releases/tag/v0.4.0) [Compare Source](https://github.com/bufbuild/protovalidate-go/compare/v0.3.4...v0.4.0) #### What's Changed - Fix bug where cel expression cannot compile for fields of type google.protobuf.Any by [@​oliversun9](https://github.com/oliversun9) in [https://github.com/bufbuild/protovalidate-go/pull/65](https://github.com/bufbuild/protovalidate-go/pull/65) - Link to connect/validate-go by [@​emcfarlane](https://github.com/emcfarlane) in [https://github.com/bufbuild/protovalidate-go/pull/66](https://github.com/bufbuild/protovalidate-go/pull/66) - Run CI on Go 1.19 by [@​akshayjshah](https://github.com/akshayjshah) in [https://github.com/bufbuild/protovalidate-go/pull/72](https://github.com/bufbuild/protovalidate-go/pull/72) - Use make lint in CI instead of golangci-lint action by [@​oliversun9](https://github.com/oliversun9) in [https://github.com/bufbuild/protovalidate-go/pull/70](https://github.com/bufbuild/protovalidate-go/pull/70) - Add isIpPrefix by [@​higebu](https://github.com/higebu) in [https://github.com/bufbuild/protovalidate-go/pull/53](https://github.com/bufbuild/protovalidate-go/pull/53) #### New Contributors - [@​higebu](https://github.com/higebu) made their first contribution in [https://github.com/bufbuild/protovalidate-go/pull/53](https://github.com/bufbuild/protovalidate-go/pull/53) **Full Changelog**: bufbuild/protovalidate-go@v0.3.4...v0.4.0 ### [`v0.3.4`](https://github.com/bufbuild/protovalidate-go/releases/tag/v0.3.4) [Compare Source](https://github.com/bufbuild/protovalidate-go/compare/v0.3.3...v0.3.4) #### What's Changed - Make DefaultResolver public by [@​oliversun9](https://github.com/oliversun9) in [https://github.com/bufbuild/protovalidate-go/pull/59](https://github.com/bufbuild/protovalidate-go/pull/59) - Update minimum required Go version from 1.18 to 1.19 by [@​nicksnyder](https://github.com/nicksnyder) in [https://github.com/bufbuild/protovalidate-go/pull/62](https://github.com/bufbuild/protovalidate-go/pull/62) - Fix ignore path for resolver.go by [@​nicksnyder](https://github.com/nicksnyder) in [https://github.com/bufbuild/protovalidate-go/pull/63](https://github.com/bufbuild/protovalidate-go/pull/63) #### New Contributors - [@​nicksnyder](https://github.com/nicksnyder) made their first contribution in [https://github.com/bufbuild/protovalidate-go/pull/62](https://github.com/bufbuild/protovalidate-go/pull/62) **Full Changelog**: bufbuild/protovalidate-go@v0.3.3...v0.3.4 ### [`v0.3.3`](https://github.com/bufbuild/protovalidate-go/releases/tag/v0.3.3) [Compare Source](https://github.com/bufbuild/protovalidate-go/compare/v0.3.2...v0.3.3) #### What's Changed - Update benchmarks by [@​rodaine](https://github.com/rodaine) in [https://github.com/bufbuild/protovalidate-go/pull/50](https://github.com/bufbuild/protovalidate-go/pull/50) - Bug: transitive field CEL expressions fail to resolve types during type checking by [@​rodaine](https://github.com/rodaine) in [https://github.com/bufbuild/protovalidate-go/pull/51](https://github.com/bufbuild/protovalidate-go/pull/51) - Fix loading field message when dependency is more than one step by [@​oliversun9](https://github.com/oliversun9) in [https://github.com/bufbuild/protovalidate-go/pull/54](https://github.com/bufbuild/protovalidate-go/pull/54) - Bump github.com/google/cel-go from 0.18.0 to 0.18.1 by [@​dependabot](https://github.com/dependabot) in [https://github.com/bufbuild/protovalidate-go/pull/55](https://github.com/bufbuild/protovalidate-go/pull/55) - Make constraint resolution more flexible to different concrete extension types by [@​rodaine](https://github.com/rodaine) in [https://github.com/bufbuild/protovalidate-go/pull/57](https://github.com/bufbuild/protovalidate-go/pull/57) - Move package `celext` out of internal by [@​oliversun9](https://github.com/oliversun9) in [https://github.com/bufbuild/protovalidate-go/pull/56](https://github.com/bufbuild/protovalidate-go/pull/56) #### New Contributors - [@​oliversun9](https://github.com/oliversun9) made their first contribution in [https://github.com/bufbuild/protovalidate-go/pull/54](https://github.com/bufbuild/protovalidate-go/pull/54) **Full Changelog**: bufbuild/protovalidate-go@v0.3.2...v0.3.3 ### [`v0.3.2`](https://github.com/bufbuild/protovalidate-go/releases/tag/v0.3.2) [Compare Source](https://github.com/bufbuild/protovalidate-go/compare/v0.3.1...v0.3.2) #### What's Changed - Build validator copy cache on write by [@​emcfarlane](https://github.com/emcfarlane) in [https://github.com/bufbuild/protovalidate-go/pull/31](https://github.com/bufbuild/protovalidate-go/pull/31) - Bump github.com/google/cel-go from 0.17.4 to 0.17.6 by [@​dependabot](https://github.com/dependabot) in [https://github.com/bufbuild/protovalidate-go/pull/39](https://github.com/bufbuild/protovalidate-go/pull/39) - Bump github.com/google/cel-go from 0.17.6 to 0.18.0 by [@​dependabot](https://github.com/dependabot) in [https://github.com/bufbuild/protovalidate-go/pull/42](https://github.com/bufbuild/protovalidate-go/pull/42) - Bump buf.build/gen/go/bufbuild/protovalidate/protocolbuffers/go from 1.31.0-20230824200731-b9b8148056b9.1 to 1.31.0-20230830185350-7a34d6557349.1 by [@​dependabot](https://github.com/dependabot) in [https://github.com/bufbuild/protovalidate-go/pull/41](https://github.com/bufbuild/protovalidate-go/pull/41) - Bypass deprecation lint warning by [@​akshayjshah](https://github.com/akshayjshah) in [https://github.com/bufbuild/protovalidate-go/pull/45](https://github.com/bufbuild/protovalidate-go/pull/45) - Bump actions/checkout from 3 to 4 by [@​dependabot](https://github.com/dependabot) in [https://github.com/bufbuild/protovalidate-go/pull/46](https://github.com/bufbuild/protovalidate-go/pull/46) - Cleanup: replace deprecated OptCheckStringFormat by [@​rodaine](https://github.com/rodaine) in [https://github.com/bufbuild/protovalidate-go/pull/48](https://github.com/bufbuild/protovalidate-go/pull/48) - Conformance: support for multiple uniques by [@​rodaine](https://github.com/rodaine) in [https://github.com/bufbuild/protovalidate-go/pull/49](https://github.com/bufbuild/protovalidate-go/pull/49) #### New Contributors - [@​emcfarlane](https://github.com/emcfarlane) made their first contribution in [https://github.com/bufbuild/protovalidate-go/pull/31](https://github.com/bufbuild/protovalidate-go/pull/31) - [@​akshayjshah](https://github.com/akshayjshah) made their first contribution in [https://github.com/bufbuild/protovalidate-go/pull/45](https://github.com/bufbuild/protovalidate-go/pull/45) **Full Changelog**: bufbuild/protovalidate-go@v0.3.1...v0.3.2 </details> <details> <summary>cerbos/cloud-api (github.com/cerbos/cloud-api)</summary> ### [`v0.1.9`](https://github.com/cerbos/cloud-api/compare/v0.1.8...v0.1.9) [Compare Source](https://github.com/cerbos/cloud-api/compare/v0.1.8...v0.1.9) </details> <details> <summary>go-logr/zapr (github.com/go-logr/zapr)</summary> ### [`v1.3.0`](https://github.com/go-logr/zapr/releases/tag/v1.3.0) [Compare Source](https://github.com/go-logr/zapr/compare/v1.2.4...v1.3.0) This release adds [support for slog](https://github.com/go-logr/logr#slog-interoperability). zapr implements `slogr.SlogSink` and therefore can be used through [`slogr.NewSlogHandler`](https://pkg.go.dev/github.com/go-logr/[email protected]/slogr#NewSlogHandler) as backend for slog. #### What's Changed - Added dependabot by [@​Neo2308](https://github.com/Neo2308) in [https://github.com/go-logr/zapr/pull/63](https://github.com/go-logr/zapr/pull/63) - Updated min supported version to go 1.18 by [@​Neo2308](https://github.com/Neo2308) in [https://github.com/go-logr/zapr/pull/62](https://github.com/go-logr/zapr/pull/62) - update linter config and fix issues by [@​pohly](https://github.com/pohly) in [https://github.com/go-logr/zapr/pull/61](https://github.com/go-logr/zapr/pull/61) - Bump github.com/go-logr/logr from 1.2.4 to 1.3.0 by [@​dependabot](https://github.com/dependabot) in [https://github.com/go-logr/zapr/pull/71](https://github.com/go-logr/zapr/pull/71) - support slog by [@​pohly](https://github.com/pohly) in [https://github.com/go-logr/zapr/pull/60](https://github.com/go-logr/zapr/pull/60) *** - Bump github.com/stretchr/testify from 1.8.0 to 1.8.4 by [@​dependabot](https://github.com/dependabot) in [https://github.com/go-logr/zapr/pull/65](https://github.com/go-logr/zapr/pull/65) - Bump actions/checkout from 2 to 3 by [@​dependabot](https://github.com/dependabot) in [https://github.com/go-logr/zapr/pull/66](https://github.com/go-logr/zapr/pull/66) - Bump actions/setup-go from 2 to 4 by [@​dependabot](https://github.com/dependabot) in [https://github.com/go-logr/zapr/pull/67](https://github.com/go-logr/zapr/pull/67) - Bump golangci/golangci-lint-action from 2 to 3 by [@​dependabot](https://github.com/dependabot) in [https://github.com/go-logr/zapr/pull/68](https://github.com/go-logr/zapr/pull/68) - Bump actions/checkout from 3 to 4 by [@​dependabot](https://github.com/dependabot) in [https://github.com/go-logr/zapr/pull/69](https://github.com/go-logr/zapr/pull/69) - Bump go.uber.org/zap from 1.24.0 to 1.25.0 by [@​dependabot](https://github.com/dependabot) in [https://github.com/go-logr/zapr/pull/64](https://github.com/go-logr/zapr/pull/64) - Bump go.uber.org/zap from 1.25.0 to 1.26.0 by [@​dependabot](https://github.com/dependabot) in [https://github.com/go-logr/zapr/pull/70](https://github.com/go-logr/zapr/pull/70) #### New Contributors - [@​Neo2308](https://github.com/Neo2308) made their first contribution in [https://github.com/go-logr/zapr/pull/63](https://github.com/go-logr/zapr/pull/63) - [@​dependabot](https://github.com/dependabot) made their first contribution in [https://github.com/go-logr/zapr/pull/65](https://github.com/go-logr/zapr/pull/65) **Full Changelog**: go-logr/zapr@v1.2.4...v1.3.0 </details> <details> <summary>google/cel-go (github.com/google/cel-go)</summary> ### [`v0.18.1`](https://github.com/google/cel-go/releases/tag/v0.18.1) [Compare Source](https://github.com/google/cel-go/compare/v0.18.0...v0.18.1) #### What's Changed - Add support for a relative offset within ast.SourceInfo by \[[#​836](https://github.com/google/cel-go/issues/836)] - Fix last optional element to be retained as an optional index while folding \[[#​841](https://github.com/google/cel-go/issues/841)] - Fix deprecation notice for string format validation \[[#​840](https://github.com/google/cel-go/issues/840)] - Update cel-spec and enable wrappers conformance tests \[[#​842](https://github.com/google/cel-go/issues/842)] - refactor: remove lexer and parser pools \[[#​838](https://github.com/google/cel-go/issues/838)] #### New Contributors - [@​TulgaCG](https://github.com/TulgaCG) made their first contribution in [https://github.com/google/cel-go/pull/835](https://github.com/google/cel-go/pull/835) - [@​aimuz](https://github.com/aimuz) made their first contribution in [https://github.com/google/cel-go/pull/838](https://github.com/google/cel-go/pull/838) **Full Changelog**: google/cel-go@v0.18.0...v0.18.1 ### [`v0.18.0`](https://github.com/google/cel-go/releases/tag/v0.18.0) [Compare Source](https://github.com/google/cel-go/compare/v0.17.7...v0.18.0) #### Features The latest release of CEL introduces validators ([#​775](https://github.com/google/cel-go/issues/775)) and optimizers ([#​804](https://github.com/google/cel-go/issues/804), [#​827](https://github.com/google/cel-go/issues/827)) and migrates the core CEL internals off of the protobuf expression and type representations ([#​789](https://github.com/google/cel-go/issues/789)). - String format validator \[[#​775](https://github.com/google/cel-go/issues/775)] - Create a Function that Reverses a String \[[#​796](https://github.com/google/cel-go/issues/796)] - Introduce pre-order / post-order visitor pattern \[[#​813](https://github.com/google/cel-go/issues/813)] - Add Libraries() function to Env \[[#​822](https://github.com/google/cel-go/issues/822)] - Static optimizer for constant folding \[[#​804](https://github.com/google/cel-go/issues/804)] - Inlining optimizer \[[#​827](https://github.com/google/cel-go/issues/827)] - FindStructTypeFields support for types.Provider \[[#​814](https://github.com/google/cel-go/issues/814)] #### Breaking Changes The following PR changes the API signature of the `checker.AstNode` method `Expr` to return an `ast.Expr`. - Migrate the checker.Coster to the ast.Expr \[[#​798](https://github.com/google/cel-go/issues/798)] #### Fixes - Nil safety checks for cel.Ast \[[#​784](https://github.com/google/cel-go/issues/784)] - Fix cost estimates to propagate result sizes \[[#​787](https://github.com/google/cel-go/issues/787)] - Catch invalid literals created from expression factories \[[#​810](https://github.com/google/cel-go/issues/810)] - Ensure stable ordering of overload candidates \[[#​817](https://github.com/google/cel-go/issues/817)] - Clarify replace with/by empty string \[[#​820](https://github.com/google/cel-go/issues/820)] - Fix functional exemptions for homogeneous literal checks \[[#​832](https://github.com/google/cel-go/issues/832)] - Fix logical operator folding that only involve literals \[[#​833](https://github.com/google/cel-go/issues/833)] - Upgrade go-genproto to latest \[[#​831](https://github.com/google/cel-go/issues/831)] #### New Contributors - [@​bboogler](https://github.com/bboogler) made their first contribution in [https://github.com/google/cel-go/pull/796](https://github.com/google/cel-go/pull/796) **Full Changelog**: google/cel-go@v0.17.1...v0.18.0 ### [`v0.17.7`](https://github.com/google/cel-go/releases/tag/v0.17.7) [Compare Source](https://github.com/google/cel-go/compare/v0.17.6...v0.17.7) #### What's Changed - Backport [#​850](https://github.com/google/cel-go/issues/850): Sets cost estimation and tracking options \[[#​852](https://github.com/google/cel-go/issues/852)] **Full Changelog**: google/cel-go@v0.17.6...v0.17.7 </details> <details> <summary>gorilla/mux (github.com/gorilla/mux)</summary> ### [`v1.8.1`](https://github.com/gorilla/mux/releases/tag/v1.8.1) [Compare Source](https://github.com/gorilla/mux/compare/v1.8.0...v1.8.1) #### What's Changed - build: CircleCI 2.1 + build matrix by [@​elithrar](https://github.com/elithrar) in [https://github.com/gorilla/mux/pull/595](https://github.com/gorilla/mux/pull/595) - Include "404" and "405" in the docs by [@​Jille](https://github.com/Jille) in [https://github.com/gorilla/mux/pull/602](https://github.com/gorilla/mux/pull/602) - docs: update README w.r.t new maintainer ask by [@​elithrar](https://github.com/elithrar) in [https://github.com/gorilla/mux/pull/660](https://github.com/gorilla/mux/pull/660) - regexp: use iota instead of hardcoded values for regexType\* by [@​michaelgrigoryan25](https://github.com/michaelgrigoryan25) in [https://github.com/gorilla/mux/pull/679](https://github.com/gorilla/mux/pull/679) - Fix `authenticationMiddleware` initialization in the `README.md` file by [@​amustaque97](https://github.com/amustaque97) in [https://github.com/gorilla/mux/pull/693](https://github.com/gorilla/mux/pull/693) - Update README.md by [@​coreydaley](https://github.com/coreydaley) in [https://github.com/gorilla/mux/pull/713](https://github.com/gorilla/mux/pull/713) - \[GPT-95] Update go version, add tools for verification and testing by [@​apoorvajagtap](https://github.com/apoorvajagtap) in [https://github.com/gorilla/mux/pull/718](https://github.com/gorilla/mux/pull/718) - Delete release-drafter.yml by [@​coreydaley](https://github.com/coreydaley) in [https://github.com/gorilla/mux/pull/719](https://github.com/gorilla/mux/pull/719) - Delete stale.yml by [@​coreydaley](https://github.com/coreydaley) in [https://github.com/gorilla/mux/pull/720](https://github.com/gorilla/mux/pull/720) - Delete AUTHORS by [@​coreydaley](https://github.com/coreydaley) in [https://github.com/gorilla/mux/pull/721](https://github.com/gorilla/mux/pull/721) - Update LICENSE by [@​coreydaley](https://github.com/coreydaley) in [https://github.com/gorilla/mux/pull/722](https://github.com/gorilla/mux/pull/722) - Updated the logo in README.md by [@​shamkarthik](https://github.com/shamkarthik) in [https://github.com/gorilla/mux/pull/724](https://github.com/gorilla/mux/pull/724) - Update LICENSE by [@​coreydaley](https://github.com/coreydaley) in [https://github.com/gorilla/mux/pull/723](https://github.com/gorilla/mux/pull/723) - Update issues.yml by [@​coreydaley](https://github.com/coreydaley) in [https://github.com/gorilla/mux/pull/726](https://github.com/gorilla/mux/pull/726) - Update issues.yml by [@​coreydaley](https://github.com/coreydaley) in [https://github.com/gorilla/mux/pull/727](https://github.com/gorilla/mux/pull/727) - run go fmt with Go 1.20 by [@​shogo82148](https://github.com/shogo82148) in [https://github.com/gorilla/mux/pull/725](https://github.com/gorilla/mux/pull/725) - Fix `Single Page Application` example in `README.md` file by [@​amustaque97](https://github.com/amustaque97) in [https://github.com/gorilla/mux/pull/678](https://github.com/gorilla/mux/pull/678) - \[BUG] Inconsistent HTTP status code on query mismatch by [@​soheilrt](https://github.com/soheilrt) in [https://github.com/gorilla/mux/pull/712](https://github.com/gorilla/mux/pull/712) - Clarify documentation examples of Route methods by [@​andrew-werdna](https://github.com/andrew-werdna) in [https://github.com/gorilla/mux/pull/672](https://github.com/gorilla/mux/pull/672) - changed the routeVariables text content. by [@​sumanpaikdev](https://github.com/sumanpaikdev) in [https://github.com/gorilla/mux/pull/708](https://github.com/gorilla/mux/pull/708) - Add GetVarNames() by [@​eh-steve](https://github.com/eh-steve) in [https://github.com/gorilla/mux/pull/676](https://github.com/gorilla/mux/pull/676) - fix SPA handler in README.md by [@​sy9](https://github.com/sy9) in [https://github.com/gorilla/mux/pull/733](https://github.com/gorilla/mux/pull/733) - update GitHub workflows by [@​coreydaley](https://github.com/coreydaley) in [https://github.com/gorilla/mux/pull/734](https://github.com/gorilla/mux/pull/734) #### New Contributors - [@​Jille](https://github.com/Jille) made their first contribution in [https://github.com/gorilla/mux/pull/602](https://github.com/gorilla/mux/pull/602) - [@​michaelgrigoryan25](https://github.com/michaelgrigoryan25) made their first contribution in [https://github.com/gorilla/mux/pull/679](https://github.com/gorilla/mux/pull/679) - [@​amustaque97](https://github.com/amustaque97) made their first contribution in [https://github.com/gorilla/mux/pull/693](https://github.com/gorilla/mux/pull/693) - [@​coreydaley](https://github.com/coreydaley) made their first contribution in [https://github.com/gorilla/mux/pull/713](https://github.com/gorilla/mux/pull/713) - [@​apoorvajagtap](https://github.com/apoorvajagtap) made their first contribution in [https://github.com/gorilla/mux/pull/718](https://github.com/gorilla/mux/pull/718) - [@​shamkarthik](https://github.com/shamkarthik) made their first contribution in [https://github.com/gorilla/mux/pull/724](https://github.com/gorilla/mux/pull/724) - [@​shogo82148](https://github.com/shogo82148) made their first contribution in [https://github.com/gorilla/mux/pull/725](https://github.com/gorilla/mux/pull/725) - [@​soheilrt](https://github.com/soheilrt) made their first contribution in [https://github.com/gorilla/mux/pull/712](https://github.com/gorilla/mux/pull/712) - [@​andrew-werdna](https://github.com/andrew-werdna) made their first contribution in [https://github.com/gorilla/mux/pull/672](https://github.com/gorilla/mux/pull/672) - [@​sumanpaikdev](https://github.com/sumanpaikdev) made their first contribution in [https://github.com/gorilla/mux/pull/708](https://github.com/gorilla/mux/pull/708) - [@​eh-steve](https://github.com/eh-steve) made their first contribution in [https://github.com/gorilla/mux/pull/676](https://github.com/gorilla/mux/pull/676) - [@​sy9](https://github.com/sy9) made their first contribution in [https://github.com/gorilla/mux/pull/733](https://github.com/gorilla/mux/pull/733) **Full Changelog**: gorilla/mux@v1.8.0...v1.8.1 </details> <details> <summary>jackc/pgx (github.com/jackc/pgx/v5)</summary> ### [`v5.5.0`](https://github.com/jackc/pgx/compare/v5.4.3...v5.5.0) [Compare Source](https://github.com/jackc/pgx/compare/v5.4.3...v5.5.0) </details> <details> <summary>lestrrat-go/jwx (github.com/lestrrat-go/jwx/v2)</summary> ### [`v2.0.16`](https://github.com/lestrrat-go/jwx/releases/tag/v2.0.16) [Compare Source](https://github.com/lestrrat-go/jwx/compare/v2.0.15...v2.0.16) v2.0.16 31 Oct 2023 [Security] * [jws] ECDSA signature verification requires us to check if the signature is of the desired length of bytes, but this check that used to exist before had been removed in #​65, resulting in certain malformed signatures to pass verification. One of the ways this could happen if R is a 31 byte integer and S is 32 byte integer, both containing the correct signature values, but R is not zero-padded. Correct = R: [ 0 , ... ] (32 bytes) S: [ ... ] (32 bytes) Wrong = R: [ ... ] (31 bytes) S: [ ... ] (32 bytes) In order for this check to pass, you would still need to have all 63 bytes populated with the correct signature. The only modification a bad actor may be able to do is to add one more byte at the end, in which case the first 32 bytes (including what would have been S's first byte) is used for R, and S would contain the rest. But this will only result in the verification to fail. Therefore this in itself should not pose any security risk, albeit allowing some illegally formated messages to be verified. * [jwk] `jwk.Key` objects now have a `Validate()` method to validate the data stored in the keys. However, this still does not necessarily mean that the key's are valid for use in cryptographic operations. If `Validate()` is successful, it only means that the keys are in the right _format_, including the presence of required fields and that certain fields have proper length, etc. [New Features] * [jws] Added `jws.WithValidateKey()` to force calling `key.Validate()` before signing or verification. * [jws] `jws.Sign()` now returns a special type of error that can hold the individual errors from the signers. The stringification is still the same as before to preserve backwards compatibility. * [jwk] Added `jwk.IsKeyValidationError` that checks if an error is an error from `key.Validate()`. [Bug Fixes] * [jwt] `jwt.ParseInsecure()` was running verification if you provided a key via `jwt.WithKey()` or `jwt.WithKeySet()` (#​1007) </details> <details> <summary>pterm/pterm (github.com/pterm/pterm)</summary> ### [`v0.12.70`](https://github.com/pterm/pterm/releases/tag/v0.12.70): Heatmap Printer 🎉 [Compare Source](https://github.com/pterm/pterm/compare/v0.12.69...v0.12.70) <!-- Release notes generated using configuration in .github/release.yml at master --> #### What's Changed ##### Exciting New Features 🎉 - Feature: Default value for interactive text input by [@​KarolosLykos](https://github.com/KarolosLykos) in [https://github.com/pterm/pterm/pull/577](https://github.com/pterm/pterm/pull/577) - Added a heatmap printer by [@​floaust](https://github.com/floaust) in [https://github.com/pterm/pterm/pull/487](https://github.com/pterm/pterm/pull/487) <img width="800" src="https://github.com/pterm/pterm/assets/56639481/c994c395-3b94-4b27-af20-4ae5fd6fc0be" /> ##### Fixes 🔧 * fix(heatmap): fix bug legend was not fully boxed by @​floau[https://github.com/pterm/pterm/pull/583](https://github.com/pterm/pterm/pull/583)ll/583 * fix(heatmap): fix bug legend was too long by @​floau[https://github.com/pterm/pterm/pull/585](https://github.com/pterm/pterm/pull/585)ll/585 **Full Changelog**: pterm/pterm@v0.12.69...v0.12.70 </details> <details> <summary>twmb/franz-go (github.com/twmb/franz-go)</summary> ### [`v1.15.2`](https://github.com/twmb/franz-go/blob/HEAD/CHANGELOG.md#v1152) [Compare Source](https://github.com/twmb/franz-go/compare/v1.15.1...v1.15.2) \=== This patch release fixes two bugs and changes Mark functions to be no-ops when not using AutoCommitMarks to avoid confusion. This also includes a minor commit further improving the sticky balancer. See the commits for more details. - [`72778cb`](https://github.com/twmb/franz-go/commit/72778cb) **behavior change** kgo: no-op mark functions when not using AutoCommitMarks - [`e209bb6`](https://github.com/twmb/franz-go/commit/e209bb6) **bugfix** kgo: pin AddPartitionsToTxn to v3 when using one transaction - [`36b4437`](https://github.com/twmb/franz-go/commit/36b4437) sticky: further improvements - [`af5bc1f`](https://github.com/twmb/franz-go/commit/af5bc1f) **bugfix** kgo: be sure to use topics when other topics are paused </details> <details> <summary>vektra/mockery (github.com/vektra/mockery/v2)</summary> ### [`v2.36.1`](https://github.com/vektra/mockery/releases/tag/v2.36.1) [Compare Source](https://github.com/vektra/mockery/compare/v2.36.0...v2.36.1) #### Changelog - [`b648c23`](https://github.com/vektra/mockery/commit/b648c23) Add additional test - [`0310201`](https://github.com/vektra/mockery/commit/0310201) Add fix for showconfig command - [`d3515d1`](https://github.com/vektra/mockery/commit/d3515d1) Fix bug with sub-package inheritance - [`77064ad`](https://github.com/vektra/mockery/commit/77064ad) Fix config bug where mockery crashes when package map is nil - [`5978bc5`](https://github.com/vektra/mockery/commit/5978bc5) Fix test with config initialization - [`deb4860`](https://github.com/vektra/mockery/commit/deb4860) Merge pull request [#​730](https://github.com/vektra/mockery/issues/730) from LandonTClipp/issue\_726 - [`e86d230`](https://github.com/vektra/mockery/commit/e86d230) Simplifying some config in interface copying code - [`726d76c`](https://github.com/vektra/mockery/commit/726d76c) Update running.md - [`2dd8f00`](https://github.com/vektra/mockery/commit/2dd8f00) Use gotestsum for better testing output </details> <details> <summary>cznic/sqlite (modernc.org/sqlite)</summary> ### [`v1.27.0`](https://gitlab.com/cznic/sqlite/compare/v1.26.0...v1.27.0) [Compare Source](https://gitlab.com/cznic/sqlite/compare/v1.26.0...v1.27.0) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "before 4am on Monday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://github.com/renovatebot/renovate/discussions) if that's undesired. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/cerbos/cerbos). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMS41IiwidXBkYXRlZEluVmVyIjoiMzcuNDYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==--> --------- Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Signed-off-by: Oğuzhan Durgun <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Oğuzhan Durgun <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #588
Summary of Changes
README.md
file.PS: If you want to verify how I'm able to reproduce the issue and tried my fix. Here is a link of my local PR where I have configured
Github Action
to execute same test cases on all the platforms i.e.ubuntu
,macos
andwindows
.PR link: amustaque97#1