-
-
Notifications
You must be signed in to change notification settings - Fork 10
Comparing changes
Open a pull request
base repository: NpgsqlRest/NpgsqlRest
base: 79ed180
head repository: NpgsqlRest/NpgsqlRest
compare: e036f3d
- 5 commits
- 22 files changed
- 1 contributor
Commits on Jun 13, 2026
-
fix(docker): JIT image — copy NpgsqlRest.Common shared source; CI: st…
…op publishing on PRs The JIT Docker build (docker/Dockerfile.jit) compiles from source inside the container, and NpgsqlRest.csproj now compiles + global-uses the shared source in NpgsqlRest.Common/ (<Compile Include="..\NpgsqlRest.Common\*.cs"/>). That folder was never copied into the build context, so the publish failed with CS0234 'NpgsqlRest.Common'. Added the COPY; validated by building the image locally. (The other Docker variants copy a prebuilt binary, so they were fine.) CI: removed the pull_request trigger so the publish/release pipeline no longer runs on every PR (dependabot PRs were flooding Actions with failing publish runs — PRs can't publish anyway, no secrets). TEMP (this commit only): build-test-publish is if:false and build-docker-jit is detached from create-release, so this push runs ONLY build-docker-jit to push the missing v3.17.0-jit / latest-jit image. To be reverted once green.
Configuration menu - View commit details
-
Copy full SHA for 53ca097 - Browse repository at this point
Copy the full SHA 53ca097View commit details -
docs(changelog): mark v3.17.0 released (2026-06-13); CI: restore full…
… release workflow Re-enables the full release chain (build-test-publish no longer if:false; build-docker-jit needs create-release again) now that the v3.17.0-jit image is published. The pull_request trigger stays removed, so the publish/release pipeline runs only on push to master + manual dispatch (no PR/dependabot noise).
Configuration menu - View commit details
-
Copy full SHA for 1f6e407 - Browse repository at this point
Copy the full SHA 1f6e407View commit details
Commits on Jun 23, 2026
-
fix(http-types): fire one outbound call per distinct HTTP type (v3.17.1)
A DB-function composite parameter is expanded into one parameter per field, each carrying the same TypeDescriptor.CustomType. The per-request list of HTTP types therefore held the type name once per field, and InvokeAllAsync fired InvokeAsync once per entry — making N identical outbound calls (N = field count) while the fill loop resolved handlers by distinct type name. SQL-file endpoints kept the type as a single composite parameter and were unaffected. Guard the firing loop to request each distinct HTTP type once, reusing the dictionary the fill loop already keys on. Preserves the established one-call-per-distinct-type contract. Adds two regression tests that count actual outbound calls via a WireMock callback: a 6-field type fires exactly 1 call (was 6), and two distinct types fire 1 call each. Verified failing pre-fix (found 6). See changelog/v3.17.1.md.
Configuration menu - View commit details
-
Copy full SHA for bf78434 - Browse repository at this point
Copy the full SHA bf78434View commit details -
feat(http-types): response caching via @cache; parse directives after…
… headers (v3.18.0) Add opt-in HTTP Custom Type response caching with a @cache <interval> type-comment directive (alongside @timeout/@retry_delay): - GET-only (warn + ignore on non-GET); success-only storage so a transient upstream failure is never pinned for the TTL. - New HttpResponseCache: in-memory store with Lazy<Task> stampede coalescing (one outbound call per key), max-entries cap, prune timer. - Cache key = method + resolved URL + content-type + headers + body. - HttpClientOptions.CacheEnabled (global kill switch), MaxCacheEntries, CachePruneIntervalSeconds; full config round-trip wired (Builder, appsettings.json, ConfigTemplate, ConfigDefaults, ConfigSchemaGenerator). Also fix: @timeout/@retry_delay/@cache are now parsed both before the request line AND after the headers. Previously only the leading position was recognized, so a directive after the headers (as the docs showed) was silently ignored. Real headers (e.g. Cache-Control) are unaffected. Folds in the v3.17.1 duplicate-outbound-call fix: version.txt and npm/package.json bumped to 3.18.0, changelog renamed v3.17.1 -> v3.18.0 covering both the feature and the fix. Tests: parse-level @cache forms + both-placement directive tests; integration tests for cache hit, 6-field dedup+cache, error-not-cached, POST-ignored, TTL expiry. Full suite green (2284).
Configuration menu - View commit details
-
Copy full SHA for 9f67d87 - Browse repository at this point
Copy the full SHA 9f67d87View commit details -
Configuration menu - View commit details
-
Copy full SHA for e036f3d - Browse repository at this point
Copy the full SHA e036f3dView commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 79ed180...e036f3d