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

Temple v2 post audit #842

Merged
merged 58 commits into from
Aug 23, 2023
Merged

Temple v2 post audit #842

merged 58 commits into from
Aug 23, 2023

Conversation

frontier159
Copy link
Collaborator

Description

What does this PR solve?

Fixes # (issue)

Checklist

  • Code follows the style guide
  • I have performed a self-review of my own code
  • New and existing tests pass locally
  • This PR is targeting the correct branch

protocol/contracts/amo/AuraStaking.sol Outdated Show resolved Hide resolved
protocol/contracts/amo/test/RamosTestnetAuraStaking.sol Outdated Show resolved Hide resolved
protocol/test/amo/ramos.ts Show resolved Hide resolved
Copy link
Collaborator

@princetonbishop princetonbishop left a comment

Choose a reason for hiding this comment

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

left some comments

@frontier159 frontier159 merged commit 7ed36cb into stage Aug 23, 2023
5 checks passed
@frontier159 frontier159 deleted the temple-v2-post-audit branch August 23, 2023 09:03
@marshall2112 marshall2112 mentioned this pull request Aug 25, 2023
princetonbishop pushed a commit that referenced this pull request Sep 18, 2023
* fix: v2 audit: yAudit/temple-lending-report#1

* fix: v2 audit: yAudit/temple-lending-report#2

* fix: v2 audit: yAudit/temple-lending-report#3

* fix: v2 audit: yAudit/temple-lending-report#5

* fix: v2 audit: yAudit/temple-lending-report#6

* fix: v2 audit: yAudit/temple-lending-report#7

* fix: v2 audit: yAudit/temple-lending-report#9

* fix: v2 audit: yAudit/temple-lending-report#10

* fix: v2 audit: yAudit/temple-lending-report#11

* fix: v2 audit: yAudit/temple-lending-report#12

* fix: v2 audit: yAudit/temple-lending-report#14

* fix: v2 audit: yAudit/temple-lending-report#15

* fix: v2 audit: yAudit/temple-lending-report#16 yAudit/temple-lending-report#18

* fix: v2 audit: yAudit/temple-lending-report#17

* fix: v2 audit: yAudit/temple-lending-report#20

* fix: v2 audit: yAudit/temple-lending-report#22 yAudit/temple-lending-report#23

* fix: v2 audit: yAudit/temple-lending-report#24

* fix: v2 audit: yAudit/temple-lending-report#26

* fix: v2 audit: yAudit/temple-lending-report#27

* fix: v2 audit: yAudit/temple-lending-report#28

* fix: v2 audit: yAudit/temple-lending-report#29

* fix: v2 audit: yAudit/temple-lending-report#30

* fix: v2 audit: yAudit/temple-lending-report#31

* fix: v2 audit: yAudit/temple-lending-report#32

* fix: v2 audit: yAudit/temple-lending-report#35

* fix: v2 audit: yAudit/temple-lending-report#36

* fix: v2 audit: yAudit/temple-lending-report#38

* fix: v2 audit: yAudit/temple-lending-report#40

* fix: v2 audit: yAudit/temple-lending-report#41

* fix: v2 audit: yAudit/temple-lending-report#42

* fix: v2 audit: yAudit/temple-lending-report#44

* fix: v2 audit: yAudit/temple-lending-report#45 Also see PaulRBerg/prb-math#198

* fix: v2 audit: yAudit/temple-lending-report#46

* fix: v2 audit: yAudit/temple-lending-report#49

* fix: v2 audit: yAudit/temple-lending-report#48

* fix: v2 audit: yAudit/temple-lending-report#50

* fix: v2 audit: yAudit/temple-lending-report#51

* fix: v2 audit: yAudit/temple-lending-report#52

* fix: v2 audit: yAudit/temple-lending-report#53

* fix: v2 audit: yAudit/temple-lending-report#54

* fix: v2 audit: yAudit/temple-lending-report#55

* fix: v2 audit: yAudit/temple-lending-report#12  part 2

* fix: v2 audit: yAudit/temple-lending-report#5  part 2

* update dToken for gas optimisations and extra events

* fix: v2 audit: yAudit/temple-lending-report#56

* fix: v2 audit: yAudit/temple-lending-report#46 (updated to how it was addressed)

* added extra test, removed redundant error, partially reverted 6cd79a0

* updated foundry version, slither, CI

* corrected safe fork lib to be internal

* dtoken rounding

* rename postRebalanceSlippage => postRebalanceDelta

* updates after internal review

* dariox request -- add the DebtCeilingUpdated event on addStrategy

* add complex liquidation test when rescue mode/paused liquidations

* review feedback on safe guard test

* review feedback on temple base strategy
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.

2 participants