Skip to content

Commit

Permalink
Force connection reuse (#47)
Browse files Browse the repository at this point in the history
Force connection reuse (and DNS request limitation) by performing one
full request before going wide.

Resolves: #46
  • Loading branch information
Jake-Shadle authored Feb 7, 2024
1 parent cdb078b commit e8e9b41
Show file tree
Hide file tree
Showing 10 changed files with 2,895 additions and 30 deletions.
12 changes: 8 additions & 4 deletions .github/workflows/rust-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
name: Lint
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
with:
components: "clippy, rustfmt"
Expand All @@ -37,26 +37,30 @@ jobs:
features: ["--features git", "--features sparse", "--features local-builder,sparse"]
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
- run: cargo fetch
- name: cargo test build
run: cargo build --tests ${{ matrix.features }}
- name: cargo test
run: cargo test ${{ matrix.features }}
- name: Test connection reuse
if: ${{ matrix.features == '--features sparse' }}
run: |
cargo run --manifest-path tests/connect/Cargo.toml
deny-check:
name: cargo-deny
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: EmbarkStudios/cargo-deny-action@v1

publish-check:
name: Publish Check
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
- run: cargo fetch
- name: cargo publish check
Expand Down
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
/target
**/target
**/*.rs.bk
Cargo.lock
tests/flock/target
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

<!-- next-header -->
## [Unreleased] - ReleaseDate
### Fixed
- [PR#47](https://github.com/EmbarkStudios/tame-index/pull/47) fixed [#46](https://github.com/EmbarkStudios/tame-index/issues/46) by ensuring one full DNS lookup and request response roundtrip is made before going wide to ensure that excessive DNS lookups and connections are not made.

## [0.9.2] - 2024-01-21
### Changed
- [PR#45](https://github.com/EmbarkStudios/tame-index/pull/45) bumped `gix` -> 0.58
Expand Down
Loading

0 comments on commit e8e9b41

Please sign in to comment.