Skip to content

Commit 8705671

Browse files
authored
Apply suggestion from @briandoyle81
1 parent b270eeb commit 8705671

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

docs/security-best-practices.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ Some practices listed below might overlap with advice in the [Cadence Anti-Patte
1212

1313
Do not use the `access(all)` modifier on fields and functions unless absolutely necessary. Prefer `access(self)`, `access(contract)`, `access(account)`, or `access(SomeEntitlement)`. Unintentionally declaring fields or functions as `access(all)` can expose vulnerabilities in your code.
1414

15-
When writing definitions for contracts, structs, or resources, start by declaring all your fields and functions as `access(self)`. If there is a function that needs to be accessible by external code, only declare it as `access(all)` if it is a `view` function or you have a specific use case where anyone should be able to use it:
15+
When writing definitions for contracts, structs, or resources, start by declaring all your fields and functions as `access(self)`. If there is a function that needs to be accessible by external code, only declare it as `access(all)` if it is a `view` function:
1616

1717
```cadence
1818
/// Simplified Bank Account implementation

0 commit comments

Comments
 (0)