Comments (2)
We will be using a round of automatic syntax change using tools at the end, thats why not much pressure is put on the team for now. Check the style convention wiki page.
-A
Notice: This message is digitally signed, its source and integrity are verifiable.
If you mail client does not support S/MIME verification, it will display a file (smime.p7s), which includes the X.509 certificate and the signature body. Read more at Certified E-Mail with Comodo and Thunderbird in AbiusX.com
On Tir 26, 1392, at 3:32 PM, vanderaj [email protected] wrote:
Please always use braces, even if it's only one line. It's also in the PSR that this project uses.
Wrong:
if ( $foo )
blah();
Right:if ( $foo ) {
blah();
}
The wrong way is a constant source of maintenance bugs, so let's just be strict about it.—
Reply to this email directly or view it on GitHub.
from phpsec.
So like abbas pointed out, I am leaving this for now....we will apply a general guideline to the whole package in the end....
from phpsec.
Related Issues (20)
- openssl_random_pseudo_bytes fails if openssl in not available HOT 2
- Flag 'suspicious' activity, with configurable threshold for reaction
- Coding Convention contradicts itself. HOT 3
- confidentialString function uses hard-coded key HOT 108
- PROJECT NEEDS TO BE SLATED AS INACTIVE HOT 3
- [SECURITY] phpsec/random.php - CSPRNG isn't used when available
- [SECURITY] phpsec/random.php - CSPRNG isn't properly error checked
- [SECURITY] phpsec/random.php - mcrypt isn't properly error checked
- [SECURITY] phpsec/random.php - Fallback CSPRNG is not CSPRNG
- [SECURITY] phpsec/user.php - Timing attack on hash comparison
- [SECURITY] phpsec/user.php - Passwords converted to lower case before hashing
- [SECURITY] phpsec user system XSS
- update for phpsec.owasp.org? HOT 39
- Question : Session HOT 3
- doubt about error handling in session library HOT 2
- mail injection prevention HOT 1
- Binding ip address with session HOT 12
- brute force detection for time based bots HOT 3
- Modification in Rand::randstr() function required HOT 5
- Trailing whitespaces issue HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from phpsec.