Skip to content
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

feat: require admin key for maxAutoAssociations during contractUpdate #14161

Merged

Conversation

MrValioBg
Copy link
Contributor

Description:
Require admin key for maxAutoAssociations during contractUpdate

Related issue(s):

Fixes #14111

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Valentin Valkanov <valentin.valkanov@limechain.tech>
@MrValioBg MrValioBg requested review from a team and tinker-michaelj as code owners July 5, 2024 07:17
@MrValioBg MrValioBg self-assigned this Jul 5, 2024
@MrValioBg MrValioBg added this to the v0.53 milestone Jul 5, 2024
@MrValioBg MrValioBg closed this Jul 5, 2024
@MrValioBg MrValioBg reopened this Jul 5, 2024
Copy link

github-actions bot commented Jul 5, 2024

Node: Unit Test Results

  1 545 files    1 545 suites   3h 30m 19s ⏱️
115 799 tests 115 740 ✅ 59 💤 0 ❌
124 114 runs  124 055 ✅ 59 💤 0 ❌

Results for commit 5e91d09.

Copy link

github-actions bot commented Jul 5, 2024

Node: HAPI Test (Restart) Results

3 tests   3 ✅  2m 59s ⏱️
3 suites  0 💤
3 files    0 ❌

Results for commit 5e91d09.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@stoyanov-st stoyanov-st left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link

github-actions bot commented Jul 5, 2024

Node: HAPI Test (Node Death Reconnect) Results

3 tests   3 ✅  6m 39s ⏱️
3 suites  0 💤
3 files    0 ❌

Results for commit 5e91d09.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 5, 2024

Node: HAPI Test (Token) Results

 22 files   22 suites   6m 31s ⏱️
273 tests 273 ✅ 0 💤 0 ❌
351 runs  351 ✅ 0 💤 0 ❌

Results for commit 5e91d09.

Copy link

github-actions bot commented Jul 5, 2024

Node: HAPI Test (Crypto) Results

 24 files   24 suites   12m 20s ⏱️
364 tests 364 ✅ 0 💤 0 ❌
371 runs  371 ✅ 0 💤 0 ❌

Results for commit 5e91d09.

Copy link

github-actions bot commented Jul 5, 2024

Node: HAPI Test (Misc) Results

 51 files   51 suites   21m 13s ⏱️
357 tests 357 ✅ 0 💤 0 ❌
375 runs  375 ✅ 0 💤 0 ❌

Results for commit 5e91d09.

Copy link

github-actions bot commented Jul 5, 2024

Node: HAPI Test (Smart Contract) Results

 73 files   73 suites   22m 38s ⏱️
627 tests 627 ✅ 0 💤 0 ❌
680 runs  680 ✅ 0 💤 0 ❌

Results for commit 5e91d09.

Copy link

github-actions bot commented Jul 5, 2024

Node: HAPI Test (Time Consuming) Results

19 tests   19 ✅  29m 18s ⏱️
 4 suites   0 💤
 4 files     0 ❌

Results for commit 5e91d09.

Copy link
Member

@lukelee-sl lukelee-sl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@MrValioBg MrValioBg merged commit d435632 into develop Jul 15, 2024
56 of 90 checks passed
@MrValioBg MrValioBg deleted the 014111-support-max-auto-assoc-for-contract-update branch July 15, 2024 15:44
netopyr added a commit that referenced this pull request Jul 18, 2024
* develop: (528 commits)
  chore: Remove unused script (#14266)
  feat: use platform event post consensus (#14256)
  chore: Removed allowances modification on `TokenReject` (#14282)
  chore: Added newlines to property-files (#14275)
  test: add tests that shows the difference of transferToken and transferFrom from contracts (#14267)
  fix: node remove test (#14237)
  feat: Add new pbj event hasher (#14226)
  chore: updated perf nightly workflow (#14263)
  docs: Design doc for HIP-632 `isAuthorizedRaw` (#13558)
  feat: hapi tests for DAB createNode (#14228)
  feat(reconnect): log ReconnectMapStats (#14250)
  chore: add proposed `IsAssociatedSystemContractTest` test refactor (#14232)
  chore: Made StakingValidator static (#14159)
  feat: require admin key for maxAutoAssociations during contractUpdate (#14161)
  build: filter publishing by deactivating tasks (develop) (#14187)
  docs: Extend design doc for HIP-719 (#14212)
  feat: add isAssociated system contract (#14169)
  chore: File Service compiler warning fixes (#14216)
  fix: Fix casting exception for event-stream recover (#14055)
  feat: hashgraph GUI fixes (#14060)
  ...

# Conflicts:
#	hapi/build.gradle.kts
#	hapi/src/main/java/com/hedera/hapi/util/HapiUtils.java
#	hedera-node/hedera-config/src/main/java/com/hedera/node/config/data/ApiPermissionConfig.java
#	hedera-node/hedera-mono-service/src/main/java/com/hedera/node/app/service/mono/pbj/PbjConverter.java
#	hedera-node/test-clients/src/main/java/com/hedera/services/bdd/spec/transactions/TxnFactory.java
#	hedera-node/test-clients/src/main/java/com/hedera/services/bdd/spec/transactions/token/HapiTokenCreate.java
#	hedera-node/test-clients/src/main/java/com/hedera/services/bdd/spec/transactions/token/HapiTokenUpdate.java
netopyr added a commit that referenced this pull request Jul 19, 2024
* develop: (22 commits)
  ci: stop release 0.51 regression (#14290)
  refactor: 14230 Removed `uptimeData` feature. (#14231)
  chore: HAPI cloneHederaProtobufs back to main (#14298)
  feat: HIP-850: HTS mutable metadata in treasury (#14204)
  chore: Remove unused script (#14266)
  feat: use platform event post consensus (#14256)
  chore: Removed allowances modification on `TokenReject` (#14282)
  chore: Added newlines to property-files (#14275)
  test: add tests that shows the difference of transferToken and transferFrom from contracts (#14267)
  fix: node remove test (#14237)
  feat: Add new pbj event hasher (#14226)
  chore: updated perf nightly workflow (#14263)
  docs: Design doc for HIP-632 `isAuthorizedRaw` (#13558)
  feat: hapi tests for DAB createNode (#14228)
  feat(reconnect): log ReconnectMapStats (#14250)
  chore: add proposed `IsAssociatedSystemContractTest` test refactor (#14232)
  chore: Made StakingValidator static (#14159)
  feat: require admin key for maxAutoAssociations during contractUpdate (#14161)
  build: filter publishing by deactivating tasks (develop) (#14187)
  docs: Extend design doc for HIP-719 (#14212)
  ...

# Conflicts:
#	hedera-node/hedera-token-service-impl/src/main/java/com/hedera/node/app/service/token/impl/api/TokenServiceApiProvider.java
#	hedera-node/hedera-token-service-impl/src/test/java/com/hedera/node/app/service/token/impl/test/handlers/CryptoDeleteHandlerTest.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support contract w admin key setting maxAutoAssociation
3 participants