Skip to content

Commit

Permalink
Remove reject auth test from MG and its footprint
Browse files Browse the repository at this point in the history
.. Auth rejection is not implemented, this is just testing the feature
flag.
  • Loading branch information
jbesraa committed Feb 17, 2025
1 parent 410cbdc commit 7dd1846
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 163 deletions.
12 changes: 0 additions & 12 deletions .github/workflows/mg.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -96,16 +96,6 @@ jobs:
- name: Run pool-sri-test-extended_1
run: sh ./test/message-generator/test/pool-sri-test-extended_1/pool-sri-test-extended_1.sh

pool-sri-test-reject-auth:
runs-on: ubuntu-latest
steps:
- name: Checkout repository
uses: actions/checkout@v4
- name: Install cargo-llvm-cov
run: cargo install cargo-llvm-cov
- name: Run pool-sri-test-reject-auth
run: sh ./test/message-generator/test/pool-sri-test-reject-auth/pool-sri-test-reject-auth.sh

sv1-test:
runs-on: ubuntu-latest
timeout-minutes: 10
Expand Down Expand Up @@ -146,7 +136,6 @@ jobs:
pool-sri-test-close-channel,
pool-sri-test-extended_0,
pool-sri-test-extended_1,
pool-sri-test-reject-auth,
sv1-test,
translation-proxy-broke-pool,
translation-proxy-old-share
Expand All @@ -164,7 +153,6 @@ jobs:
[ "${{ needs.pool-sri-test-close-channel.result }}" != "success" ] ||
[ "${{ needs.pool-sri-test-extended_0.result }}" != "success" ] ||
[ "${{ needs.pool-sri-test-extended_1.result }}" != "success" ] ||
[ "${{ needs.pool-sri-test-reject-auth.result }}" != "success" ] ||
[ "${{ needs.sv1-test.result }}" != "success" ] ||
[ "${{ needs.translation-proxy-broke-pool.result }}" != "success" ] ||
[ "${{ needs.translation-proxy-old-share.result }}" != "success" ]; then
Expand Down
1 change: 0 additions & 1 deletion roles/pool/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -42,4 +42,3 @@ hex = "0.4.3"

[features]
test_only_allow_unencrypted = []
MG_reject_auth = []
8 changes: 0 additions & 8 deletions roles/pool/src/lib/mining_pool/message_handler.rs
Original file line number Diff line number Diff line change
Expand Up @@ -21,14 +21,6 @@ impl ParseDownstreamMiningMessages<(), NullDownstreamMiningSelector, NoRouting>
true
}

#[cfg(feature = "MG_reject_auth")]
fn is_downstream_authorized(
_self_mutex: Arc<Mutex<Self>>,
_user_identity: &binary_sv2::Str0255,
) -> Result<bool, Error> {
Ok(false)
}

fn handle_open_standard_mining_channel(
&mut self,
incoming: OpenStandardMiningChannel,
Expand Down

This file was deleted.

This file was deleted.

0 comments on commit 7dd1846

Please sign in to comment.