Recommended
Recommended
More Related Content
What's hot
What's hot (20)
Similar to What the Struts?
Similar to What the Struts? (20)
More from Joe Kutner
More from Joe Kutner (20)
Recently uploaded
Recently uploaded (20)
What the Struts?
- 1. WHAT THE STRUTS? SECURE YOUR JAVA APPS
- 2. HELLO JOE KUTNER ▸ @codefinger ▸ Java Language Owner
- 4. SECURITY IS LIKE A RUBIK'S CUBE
- 5. Security is an asymmetric problem
- 6. GOOD GUYS & GALS BAD GUYS & GALS SANITIZE INPUT 2 FACTOR AUTH WEAK-PASSWORD CHECKS ENCRYPT DATA AT REST HTTPS DEFAULT ADMIN PASSWORD MORE...
- 7. “ANY FOOL CAN THROW A STONE DOWN A WELL, BUT IT TAKES A WISE MAN TO GET IT OUT” –Chinese proverb (probably) TEXT
- 8. LET'S FOCUS ON WHAT WE CAN DO VECTORS ▸ Platform: OS, Network, CPU ▸ Client: XSS, HTTPS, CSRF, etc ▸ Social Engineering, Physical Security ▸ Application: Dependencies, Authentication, Authorization, Encryption, etc
- 9. ▸ Platform: OS, Network, CPU ▸ Client: XSS, HTTPS, CSRF, etc ▸ Social Engineering, Physical Security ▸ Application: Dependencies, Authentication, Authorization, Encryption, etc LET'S FOCUS ON WHAT WE CAN DO VECTORS
- 10. STRUTS
- 11. WHY IS THIS IMPORTANT? STRUTS IS EVERYWHERE ▸ RedMonk estimates that at least 65% of the Fortune 100 companies are actively using the Struts framework. ▸ According to the Struts website: Lockheed Martin, the IRS, Citigroup, Vodafone, Virgin Atlantic, Reader’s Digest, Office Depot, and SHOWTIME have used the framework.
- 14. Equifax has been intensely investigating the scope of the intrusion with the assistance of a leading, independent cybersecurity firm to determine what information was accessed and who has been impacted. We know that criminals exploited a U.S. website application vulnerability. The vulnerability was Apache Struts CVE-2017-5638. We continue to work with law enforcement as part of our criminal investigation, and have shared indicators of compromise with law enforcement. Equifax Statement IT WAS CVE-2017-5638 https://help.equifax.com/s/article/What-was-the-vulnerability
- 16. ADGENDA ▸ CVE-2017-5638 deep dive ▸ How it lead to the Equifax hack ▸ Why it shouldn't have happened ▸ How you can make sure it doesn't happen to you
- 17. Demo
- 18. HOW IT WORKS CVE-2017-5638 ▸ Send multipart/form-data request to Upload action ▸ Add a Content-Type header with an OGNL expression ▸ Server will execute arbitrary Java code in the expression
- 24. return LocalizedTextUtil.findText( this.getClass(), errorKey, defaultLocale, e.getMessage(), args); Contains the value from our "Content-Type" header
- 26. HOW IT HAPPENED... TIMELINE (2017) ▸ March 6: CVE-2017-5638 (S2-045) discovered ▸ March 7: Struts 2.3.32 and 2.5.10.1 released with a fix ▸ May to July: Equifax says hackers gained unauthorized access to its data ▸ July 29: Equifax discovers the hack and immediately stops the intrusion ▸ September 7: Equifax officially alerts the public
- 27. STRUTS
- 28. VERIZON DATA BREACH INVESTIGATIONS REPORT ▸ More than 70% of real-world attacks exploit a known vulnerability for which a fix is available but has not yet been applied http://www.verizonenterprise.com/verizon-insights-lab/dbir/
- 30. WHY IS THIS IMPORTANT? STRUTS IS A WAKE-UP CALL! ▸ If a vulnerability was discovered in one of the frameworks you use, would you know about it?
- 31. ARE YOU USING JACKSON? HOPEFULLY NOT VERSION 2.8.8... https://github.com/FasterXML/jackson-databind/issues/1599
- 32. PREVENT THE USE OF DEPENDENCIES WITH KNOWN VULNERABILITIES CALL TO ACTION
- 33. STRATEGIES AUTOMATE DEPENDENCY MANAGEMENT ▸ Maven Versions Plugin ▸ Snyk.io ▸ Gradle dependency.lock
- 34. Demo
- 35. MAVEN VERSIONS PLUGIN $ mvn versions:display-property-updates ... [INFO] The following version property updates are available: [INFO] ${flyway.version} ............. 3.2.1 -> 5.0.6 [INFO] ${webjars-jquery.version} ..... 2.2.4 -> 3.2.1 [INFO] ${pgjdbc.version} ............. 42.1.4 -> 42.1.4.jre7 $ mvn versions:update-properties
- 37. pom.xml <ruleset comparisonMethod="maven"> <ignoreVersions> <ignoreVersion type="regex">.*-beta.*</ignoreVersion> <ignoreVersion type="regex">.*-alpha.*</ignoreVersion> <ignoreVersion type="regex">.*-rc.*</ignoreVersion> <ignoreVersion type="regex">.*.jre7</ignoreVersion> <ignoreVersion type="regex">.*.jre6</ignoreVersion> </ignoreVersions> </ruleset>
- 38. NOT GOOD ENOUGH
- 42. https://snyk.io Continuous Vulnerability Detection & Resolution
- 45. NOT BAD
- 46. BUT STILL NOT GOOD ENOUGH
- 47. MAVEN DOESN'T LOCK TRANSITIVE DEPENDENCIES $ mvn dependency:tree [INFO] +- org.springframework.boot:spring-boot-starter-web:jar:2.0.0. [INFO] | +- org.springframework.boot:spring-boot-starter-json:jar:2. [INFO] | | +- com.fasterxml.jackson.datatype:jackson-datatype-jdk8: [INFO] | | +- com.fasterxml.jackson.datatype:jackson-datatype-jsr31 [INFO] | | - com.fasterxml.jackson.module:jackson-module-parameter [INFO] | +- org.springframework.boot:spring-boot-starter-tomcat:jar: [INFO] | | +- org.apache.tomcat.embed:tomcat-embed-core:jar:8.5.23: [INFO] | | | - org.apache.tomcat:tomcat-annotations-api:jar:8.5.2 [INFO] | | +- org.apache.tomcat.embed:tomcat-embed-el:jar:8.5.23:co [INFO] | | - org.apache.tomcat.embed:tomcat-embed-websocket:jar:8. ...
- 51. build.gradle plugins { id "nebula.dependency-lock" version "5.0.1" } dependencyLock { includeTransitives = true }
- 52. GENERATE A LOCKFILE $ ./gradlew generateLock saveLock
- 53. dependencies.lock WARNING: COMPILED CODE! DO NOT EDIT { "compile": { "aopalliance:aopalliance": { "locked": "1.0", "transitive": [ "com.google.inject:guice" ] }, "com.fasterxml.jackson.core:jackson-annotations": { "locked": "2.9.3", "transitive": [ "com.fasterxml.jackson.core:jackson-databind", "com.fasterxml.jackson.datatype:jackson-datatype-jsr310" ] }, ...
- 54. THIS IS GOOD
- 55. KNOW YOUR DEPENDENCIES TAKE ACTION! 1. Automate: mvn versions:update-properties 2. Generate dependency reports 3. Use dependency monitoring: https://snyk.io 4. Use Gradle and dependencies.lock 5. Watch NVD feeds: https://nvd.nist.gov/
- 56. STRUTS WAS NOT THE ONLY CULPRIT MANY REASONS ▸ Using dependencies with known vulnerabilities ▸ Failure to sanitize user input ▸ Lack of network segmentation ▸ Inadequate encryption of PII ▸ Ineffective intrusion detection mechanisms
- 58. LOGGING
- 59. WHAT TO LOG? ▸ Logins (Successful and Failed) ▸ Logouts ▸ Password changes ▸ User profile changes ▸ Password reset ▸ User de-registration ▸ Authorization failures ▸ Changes to access levels ▸ Operational activities (backups) ▸ Input validation failures ▸ Any sensitive operation
- 60. WHAT NOT TO LOG ▸ Session ID (hash instead) ▸ Passwords ▸ Anything sensitive
- 61. WHAT NOT TO LOG ▸ In 2012, Radu Dragusin discovered a log file on a public IEEE FTP server that contained more than 100,000 usernames and passwords ▸ Google, Apple, Microsoft, Oracle, IBM
- 62. IN ADDITION TO INFO, WARN, DEBUG, ETC HOW TO LOG ▸ SECURITY_SUCCESS ▸ SECURITY_FAILURE ▸ SECURITY_AUDIT
- 63. USE CUSTOM MARKERS LOGBACK log.warn(SecurityMarkers.SECURITY_AUDIT, "Anonymous account access. Forwarding to login"); log.error(SecurityMarkers.SECURITY_FAILURE, "Unauthorized user {} attempted admin access", user.getUsername());
- 64. YOUR LOGS SHOULD BE ABLE TO ANSWER THESE QUESTIONS LOGGING ▸ What happened? ▸ Who did it? ▸ When did it happen? ▸ How was our security circumvented? ▸ What data was viewed or modified? ▸ How can we prevent this from happening again?
- 65. DETECTION
- 66. OWASP APP SENSOR PROJECT ▸ Detect and respond to attacks from within the application
- 67. APP LAYER INTRUSION DETECTION ▸ Traditional intrusion detection systems focus on attacks below the HTTP layer ▸ They do not provide context within the application environment
- 70. WEB APP APPSENSOR HEY, THIS LOOKS WEIRD NAH, IT'S COOL
- 71. WEB APP APPSENSOR LOOKS LIKE AN ATTACK! OK, I'LL BLOCK THAT USER HEY, THIS LOOKS WEIRD
- 73. Demo
- 74. @Path("/accounts") public class AccountViewHandler { @Inject AppSensorClient ids; @GET @Path("/view") Account findAccount(@QueryParam("id") String id) throws NotAuthorizedException { User user = UserContext.getCurrentUser(); if (!user.isAuthorized(Data.Account, id)) { Event event = new Event( new User( user.getUsername()), DetectionPoints.BRUTE_FORCE_ACCOUNT); ids.addEvent(event); throw new NotAuthorizedException( "Not authorized to access this account."); } Account account = accountDao.find(id); return account; } }
- 76. TAKE ACTION INTRUSION DETECTION ▸ Log all security related actions ▸ Except secrets ▸ Monitor your logs ▸ Add Detection Points ▸ React to Detection Point Triggers
- 78. APACHE STRUTS STATEMENT ON EQUIFAX SECURITY BREACH RECOMMENDATIONS ▸ Understand which supporting frameworks and libraries are used in your software products and in which versions. Keep track of security announcements affecting this products and versions. ▸ Establish a process to quickly roll out a security fix release of your software product once supporting frameworks or libraries needs to be updated for security reasons. Best is to think in terms of hours or a few days, not weeks or months. Most breaches we become aware of are caused by failure to update software components that are known to be vulnerable for months or even years. ▸ Any complex software contains flaws. Don't build your security policy on the assumption that supporting software products are flawless, especially in terms of security vulnerabilities. ▸ Establish security layers. It is good software engineering practice to have individually secured layers behind a public-facing presentation layer such as the Apache Struts framework. A breach into the presentation layer should never empower access to significant or even all back-end information resources. ▸ Establish monitoring for unusual access patterns to your public Web resources. Nowadays there are a lot of open source and commercial products available to detect such patterns and give alerts. We recommend such monitoring as good operations practice for business critical Web-based services. https://blogs.apache.org/foundation/entry/apache-struts-statement-on-equifax
- 79. APACHE STRUTS STATEMENT ON EQUIFAX SECURITY BREACH RECOMMENDATIONS ▸ Know your dependencies ▸ Thou shall have Continuous Deployment ▸ Remember that software is insecure ▸ Thou shall have security layers ▸ Thou shall monitor for unusual patterns
- 80. GOODBYE THANK YOU ▸ @codefinger ▸ heroku.com