# Konzi

Hello! by :( Neutrino
henlo itt voltam: Valentin
na csaa: Dávid
üdvözletem: Illés

## Projekt-specifikus reszletek

URL: https://cmsch-testing.kir-dev.hu/home

Backend src: https://github.com/kir-dev/cmsch/tree/staging
Public Swagger: https://api.cmsch-testing.kir-dev.hu/swagger-ui/index.html

Ez a doksi: https://hackmd.io/@6Neqf-lbRkSH2Et8cPlY0g/ByvThJBMA/edit

Leírás: https://kir-dev.hu/project/cmsch

## Doksi hasznalata:

- Ha nagyon sok -> uj doksi + link (anyone with the link)
- Ha jegyzet, menjen [Ide](#Jegyzet)
- Ha eredmeny, eleg inline az adott teszthez
- Ha finding, akkor [Ide](#Findings)

## Tests
https://owasp.boireau.io/4-web_application_security_testing

4.1 Information Gathering
~~4.1.1 Conduct Search Engine Discovery Reconnaissance for Information Leakage~~
~~4.1.2 Fingerprint Web Server~~
~~4.1.3 Review Webserver Metafiles for Information Leakage~~
~~4.1.4 Enumerate Applications on Webserver~~
~~4.1.5 Review Web Page Content for Information Leakage~~
~~4.1.6 Identify Application Entry Points~~
~~4.1.7 Map Execution Paths Through Application~~
~~4.1.8 Fingerprint Web Application Framework~~
~~4.1.9 Fingerprint Web Application~~
~~4.1.10 Map Application Architecture~~
4.2 Configuration and Deployment Management Testing
4.2.1 Test Network Infrastructure Configuration
4.2.2 Test Application Platform Configuration
4.2.3 Test File Extensions Handling for Sensitive Information
4.2.4 Review Old Backup and Unreferenced Files for Sensitive Information
4.2.5 Enumerate Infrastructure and Application Admin Interfaces
4.2.6 Test HTTP Methods
4.2.7 Test HTTP Strict Transport Security
4.2.8 Test RIA Cross Domain Policy
4.2.9 Test File Permission
4.2.10 Test for Subdomain Takeover
4.2.11 Test Cloud Storage
4.2.12 Test for Content Security Policy
4.2.13 Test for Path Confusion
4.3 Identity Management Testing
4.3.1 Test Role Definitions
- [https://hackmd.io/@6Neqf-lbRkSH2Et8cPlY0g/B1aDS7BzA](https://hackmd.io/@6Neqf-lbRkSH2Et8cPlY0g/B1aDS7BzA)
- kell még átnézni, hogy jól kezelik-e.

4.3.2 Test User Registration Process
4.3.3 Test Account Provisioning Process
4.3.4 Testing for Account Enumeration and Guessable User Account
~~4.3.5 Testing for Weak or Unenforced Username Policy~~
- Reason: Nincsen olyen, hogy username, minden SCHACC-ból / Google-től jön.

4.4 Authentication Testing
~~4.4.1 Testing for Credentials Transported over an Encrypted Channel~~
- Reason: authentication is over HTTPS, Google SSO / SCHACC

4.4.2 Testing for Default Credentials
4.4.3 Testing for Weak Lock Out Mechanism
4.4.4 Testing for Bypassing Authentication Schema
~~4.4.5 Testing for Vulnerable Remember Password~~
- Reason: Nincs password

4.4.6 Testing for Browser Cache Weaknesses
~~4.4.7 Testing for Weak Password Policy~~
- Reason: Nincs password

~~4.4.8 Testing for Weak Security Question Answer~~
- Reason: Nincs password

~~4.4.9 Testing for Weak Password Change or Reset Functionalities~~
- Reason: Nincs password

4.4.10 Testing for Weaker Authentication in Alternative Channel
4.4.11 Testing Multi-Factor Authentication
4.5 Authorization Testing
4.5.1 Testing Directory Traversal File Include
4.5.2 Testing for Bypassing Authorization Schema
4.5.3 Testing for Privilege Escalation
4.5.4 Testing for Insecure Direct Object References
- ? tasks needs more test if its realy idor or not
4.5.5 Testing for OAuth Weaknesses
4.5.5.1 Testing for OAuth Authorization Server Weaknesses
4.5.5.2 Testing for OAuth Client Weaknesses
4.6 Session Management Testing
~~4.6.1 Testing for Session Management Schema~~
- Reason: JWT

4.6.2 Testing for Cookies Attributes
~~4.6.3 Testing for Session Fixation~~
- Reason: JWT

4.6.4 Testing for Exposed Session Variables
4.6.5 Testing for Cross Site Request Forgery
- Valentin

4.6.6 Testing for Logout Functionality
- JWT token, ha lementem, és ujra beállítom manuálisan, akkor működik, pedíg kijelentkeztem.
4.6.7 Testing Session Timeout
4.6.8 Testing for Session Puzzling
4.6.9 Testing for Session Hijacking
4.6.10 Testing JSON Web Tokens
4.6.11 Testing for Concurrent Sessions
4.7 Input Validation Testing
4.7.1 Testing for Reflected Cross Site Scripting
4.7.2 Testing for Stored Cross Site Scripting
4.7.3 Testing for HTTP Verb Tampering
4.7.4 Testing for HTTP Parameter Pollution
4.7.5 Testing for SQL Injection
~~4.7.5.1 Testing for Oracle~~
- Reason: Nincs használva
~~4.7.5.2 Testing for MySQL~~
- Reason: Nincs használva
~~4.7.5.3 Testing for SQL Server~~
- Reason: Nincs használva
~~4.7.5.4 Testing PostgreSQL~~
- Nincs nyers kérés (`.queryForObject()`, `entityManager.createQuery()`, , `@Query("...", nativeQuery = true)`)
~~4.7.5.5 Testing for MS Access~~
- Reason: Nincs használva
~~4.7.5.6 Testing for NoSQL Injection~~
- Reason: Nincs használva
4.7.5.7 Testing for ORM Injection
~~4.7.5.8 Testing for Client-side~~
- Reason: Nincs használva
~~4.7.6 Testing for LDAP Injection~~
- Reason: Nincs használva
4.7.7 Testing for XML Injection
4.7.8 Testing for SSI Injection
4.7.9 Testing for XPath Injection
4.7.10 Testing for IMAP SMTP Injection
4.7.11 Testing for Code Injection
4.7.11.1 Testing for File Inclusion
4.7.12 Testing for Command Injection
4.7.13 Testing for Format String Injection
4.7.14 Testing for Incubated Vulnerability
4.7.15 Testing for HTTP Splitting Smuggling
4.7.16 Testing for HTTP Incoming Requests
4.7.17 Testing for Host Header Injection
~~4.7.18 Testing for Server-side Template Injection~~
- Reason: nem valószínű React + Springboot stacken

4.7.19 Testing for Server-Side Request Forgery
4.7.20 Testing for Mass Assignment
4.8 Testing for Error Handling
4.8.1 Testing for Improper Error Handling
4.8.2 Testing for Stack Traces
4.9 Testing for Weak Cryptography
~~4.9.1 Testing for Weak Transport Layer Security~~
- HTTP átirányít HTTPS-re (az api. aldomainen is, feltételezhetően minden route-on és minden metódussal, habár csak néhányat néztem meg.)
- TLS 1.3
4.9.2 Testing for Padding Oracle
4.9.3 Testing for Sensitive Information Sent via Unencrypted Channels
4.9.4 Testing for Weak Encryption
4.10 Business Logic Testing
4.10.0 Introduction to Business Logic
4.10.1 Test Business Logic Data Validation
4.10.2 Test Ability to Forge Requests
4.10.3 Test Integrity Checks
4.10.4 Test for Process Timing
4.10.5 Test Number of Times a Function Can Be Used Limits
4.10.6 Testing for the Circumvention of Work Flows
4.10.7 Test Defenses Against Application Misuse
4.10.8 Test Upload of Unexpected File Types
- yes, A `backend/src/main/kotlin/hu/bme/sch/cmsch/component/task/TaskApiController.kt` fájlban definiált `api/task/submit` útvonalhoz tartozó POST-os metódus fogad egy opcionális `file` paramétert, amit új beadás esetén átad (közvetetten) a `TasksService` osztály `newSubmission` metódusának, ahol `TaskType.BOTH` esetén a fájl ellenőrzés nélkül kerül feltöltésre. Ez a file html file is lehet, amit ha valaki megnyit, akkor javascriptet lehet vele futtatni a gépükön, amivel el lehet lopni a ssession tokenjüket.

4.10.9 Test Upload of Malicious Files
4.10.10 Test Payment Functionality
4.11 Client-side Testing
4.11.1 Testing for DOM-Based Cross Site Scripting
4.11.1.1 Testing for Self DOM Based Cross-Site Scripting
4.11.2 Testing for JavaScript Execution
4.11.3 Testing for HTML Injection
4.11.4 Testing for Client-side URL Redirect
4.11.5 Testing for CSS Injection
4.11.6 Testing for Client-side Resource Manipulation
~~4.11.7 Testing Cross Origin Resource Sharing~~
- Illés
- Számos útvonalnál `Access-Control-Allow-Origin: *` header van beállítva. Amennyiben valóban szükség van CORS-ra, jobb lenne explicit felsorolni a használt oldalakat.

4.11.8 Testing for Cross Site Flashing
4.11.9 Testing for Clickjacking
~~4.11.10 Testing WebSockets~~
- Reason: Nincs használva

4.11.11 Testing Web Messaging
4.11.12 Testing Browser Storage
4.11.13 Testing for Cross Site Script Inclusion
- Dávid
- a keresőkben nincs.

4.11.14 Testing for Reverse Tabnabbing
4.12 API Testing
~~4.12.1 Testing GraphQL~~

## Bugs
- Csapatok létrehozásánál, nem akkor jelenik meg a csapat, ha a `Játszik a csapata a versenyben` be van pipálva, hanem akkor, ha a `Felhasználó készítette van bekapcsolva`

## Reports
- Informatív -> nincs vele teendő, vagy nagyon minimális
- Low -> Alacsony impact, önmagában nem kihasználható vagy nem szokás komolyan venni
- Pl. Open Redirect, Self-XSS vagy Clickjacking
- Medium -> Limitált lehetőség van arra, hogy átlépj határokat
- High -> Felhasználót tudsz impersonate-elni, vagy támadónak releváns akciót tudsz végrehajtani megfelelő jogosultság nélkül
- Critical -> Kódfuttatás, SQLi

## Jegyzet

## Findings
**Medium:** A `backend/src/main/kotlin/hu/bme/sch/cmsch/controller/admin/OneDeepEntityPage.kt` fájlban található `OneDeepEntityPage` generikus kontrollert több admin panelből elérhető kontroller is örökli. Az osztályban lévő POST-os `create` és `edit` metódusok 2 opcionális fájl paramétert (`file0`, `file1`) is kaphatnak. Ezek a fájlok ellenőrzés nélkül kerülnek mentésre az `updateEntity` metódusban. Azonban a frontend kódjában nem találtunk arra utaló jelet, hogy ezek a paraméterek használva lennének bárhol. Amennyiben valóban nincs rájuk szükség, úgy érdemes lenne ezeket eltávolítani, esetleges szerver oldali kódvégrehajtás, illetve XSS elkerülése érdekében.

**High:** A `backend/src/main/kotlin/hu/bme/sch/cmsch/component/task/TaskApiController.kt` fájlban definiált `api/task/submit` útvonalhoz tartozó POST-os metódus fogad egy opcionális `file` paramétert, amit új beadás esetén átad (közvetetten) a `TasksService` osztály `newSubmission` metódusának, ahol `TaskType.BOTH` esetén a fájl ellenőrzés nélkül kerül feltöltésre. Ez a file html file is lehet, amit ha valaki megnyit, akkor javascriptet lehet vele futtatni a gépükön, amivel el lehet lopni a ssession tokenjüket.
## Megjegyzések
- https://cmsch-testing.kir-dev.hu/profile nincs HttpOnly bekapcsolva
- Azt neztuk, hogy bmejegy integracio olyan oldalakat akar betolteni ami nem leteyik ?!
- btw tenyleg wp-admin-t kaptok hozzá, ez a bmejegy oldaláról nem teljesen szekurus :smile_cat: