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.

Sign up for GitHub

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

Jump to bottom

Merge Release/v0.7.6 into master #428

Merged
merged 81 commits into from
Jun 2, 2021
Merged

Merge Release/v0.7.6 into master #428

merged 81 commits into from
Jun 2, 2021

Conversation

minsulee2
Copy link
Member

@minsulee2 minsulee2 commented Jun 1, 2021 •
edited
Loading

0.7.6. is now ready to merge into master!

Includes:

  • Version up(v0.7.6)
  • _createApp integration with /apps config
  • Support blockchain data preservation
  • Do function sanitization logic in setFunction()
  • Add more servered characters (<, >, ', ", `, ' ') and test cases
  • Update scripts
  • Add sanitization logic (isValidRuleTree(), isValidRuleConfig()) for rule tree
  • Add p2p unittest cases
  • Fix minor bugs

minsulee2 and others added 30 commits May 18, 2021 16:01
@minsulee2
Init p2p-util.test.js
5a2b86e
@minsulee2
Merge pull request #403 from ainblockchain/release/0.7.3
833403e 
Release/0.7.3 to develop branch
@minsulee2
Add removeSocketConnectionIfExists test
59ebf29
@platfowner
Allow fid in owner config. Add _setOwnerConfig native function for te…
ddb4e7f 
…sting.
@minsulee2
Add closeSocketSafe test
e40909b
@minsulee2
Move isValidProtocolVersion to version util
d462031
@platfowner
Rename execGasAmount -> extraGasAmount.
3ceb49e
@minsulee2
Test message encapsulation
231611b
@minsulee2
Add checkTimestamp test
705a6db
@minsulee2
Underscore for inner function in util
593050b
@platfowner
Merge pull request #407 from ainblockchain/feature/platfowner/new_fea…
512d8da 
…ture

Allow function IDs (auth.fid) in owner config
@minsulee2
Leave some comments
a0541d3
@minsulee2
Add signMessage test cases
a453c91
@liayoo
Set apps owner, rule in _createApp.
66b0d53
@liayoo
Update genesis owner config.
3c15232
@liayoo
Update test cases.
1081454
@liayoo
Check shardOwner permissions, try creating app when starting a shard.
cbcc942
@minsulee2
Add sign and verify from message tests
b337639
@minsulee2
Init p2p integration test
a106787
@liayoo
Remove unnecessary owner setting.
7ae5892
@liayoo
Add app name validity check.
87a750e
@liayoo
Update db test.
601988c
@liayoo
Merge pull request #408 from ainblockchain/feature/integrate-apps-int…
db37273 
…o-createapp

Feature/integrate apps into createapp
@platfowner
Move applyFunctionChange to state-util.js.
dfa9c55
@platfowner
Add isValidOwnerConfig() util function. Extend isEmpty() util function.
59cb54e
@platfowner
Tweak waiting time in test cases.
6521dd6
@platfowner
Consider wildcard(*) and null owner config values.
868707f
@platfowner
Add isValidOwnerTree() util function.
2f91ef5
@liayoo
Introduce DATA_DIR, update default CHAIN_DIR.
ae4a7b6
@liayoo
Update deploy scripts.
5e1ecdf
pengin7384 and others added 19 commits May 31, 2021 17:43
@pengin7384
Add ZONE
8258f8a
@platfowner
Fix broken sharding test cases.
8c5525f
@platfowner
Merge pull request #423 from ainblockchain/bugfix/platfowner/bugfix
3980149 
Fix broken sharding test cases.
@platfowner
Add isValidRuleConfig(), isValidRuleTree().
0af928c
@pengin7384
Add ZONE to deploy_blockchain_incremental_gcp.sh
bfae4b9
@pengin7384
Add ZONE to deploy_monitoring_gcp.sh
6387c8a
@pengin7384
Add ZONE to reset_blockchain_gcp.sh
0405e7b
@cshcomcom
Merge pull request #422 from ainblockchain/feature/deploy-blockchain-…
bb797b1 
…gcp-script

Add ZONE
@platfowner
Apply rule config sanitization in setRule().
19bca61
@platfowner
Use common function for config sanitization.
7744329
@platfowner
Refactor isValidJsObjectForStates().
6eaaab9
@platfowner
Address minor issues (naming and typos) in tests.
3d297a4
@platfowner
Merge pull request #424 from ainblockchain/feature/platfowner/new_fea…
39be95b 
…ture

Add sanitization logic (isValidRuleTree(), isValidRuleConfig()) for rule tree
@minsulee2
Merge branch 'master' of github.com:ainblockchain/ain-blockchain into…
d7b5458 
… release/v0.7.6

Pull master hotfix
@minsulee2
Version up to 0.7.6
fd3108b
@minsulee2
Merge pull request #425 from ainblockchain/version-up/v0.7.6
dba0cb5 
Version up to 0.7.6
@minsulee2
Fix test for all node 12 users
1606c7b
@minsulee2
Swap expected and actual
df6573a
@minsulee2
Merge pull request #426 from ainblockchain/bugfix/unittest-issue-419-fix
f15ea3a 
Bugfix/unittest issue 419 fix
@minsulee2 minsulee2 added the release Release new version label Jun 1, 2021
platfowner
platfowner approved these changes Jun 1, 2021
View reviewed changes
Copy link
Member

@platfowner platfowner 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!

@liayoo
Copy link
Contributor

liayoo commented Jun 1, 2021

LGTM! Could you add the following to the release note? Thanks!

  • _createApp integration with /apps config
  • Support blockchain data preservation

@minsulee2
Copy link
Member Author

LGTM! Could you add the following to the release note? Thanks!

  • _createApp integration with /apps config
  • Support blockchain data preservation

@liayoo Thanks! updated in this PR as well!

@minsulee2
Copy link
Member Author

Thanks for the reviews! am gonna release right after merging this PR.

@minsulee2 minsulee2 merged commit 8a85c7f into master Jun 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liayoo liayoo liayoo approved these changes

@platfowner platfowner platfowner approved these changes

@shyun-comcom shyun-comcom Awaiting requested review from shyun-comcom

@cshcomcom cshcomcom Awaiting requested review from cshcomcom

Assignees
No one assigned
Labels
release Release new version
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@minsulee2 @liayoo @platfowner @pengin7384 @cshcomcom