From ec3119efc2f2056f0338a90296ea084865441e82 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 26 Jun 2025 17:32:45 +0000 Subject: [PATCH 1/3] Initial plan From 64c1d221c48d1e2a53ffaf095a2172841ae61349 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 26 Jun 2025 17:36:01 +0000 Subject: [PATCH 2/3] Initial DevSecOps GHAS demo implementation - Fixed .NET version and packages Co-authored-by: CalinL <10718943+CalinL@users.noreply.github.com> --- src/webapp01/Program.cs | 5 ++--- src/webapp01/webapp01.csproj | 4 ++-- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/src/webapp01/Program.cs b/src/webapp01/Program.cs index a04832b..3177bcf 100644 --- a/src/webapp01/Program.cs +++ b/src/webapp01/Program.cs @@ -19,8 +19,7 @@ app.UseAuthorization(); -app.MapStaticAssets(); -app.MapRazorPages() - .WithStaticAssets(); +app.UseStaticFiles(); +app.MapRazorPages(); app.Run(); diff --git a/src/webapp01/webapp01.csproj b/src/webapp01/webapp01.csproj index 9b11105..97303be 100644 --- a/src/webapp01/webapp01.csproj +++ b/src/webapp01/webapp01.csproj @@ -1,7 +1,7 @@ - net9.0 + net8.0 enable enable 7f0355f0-e3cb-4a1e-bf2d-0431db9b93f8 @@ -13,7 +13,7 @@ - + From 9c23ae1965207404ed9bd168715b4d53cb111716 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 26 Jun 2025 17:51:01 +0000 Subject: [PATCH 3/3] Add DevSecOps2 page with extended security vulnerability demonstrations Co-authored-by: CalinL <10718943+CalinL@users.noreply.github.com> --- src/webapp01/Pages/DevSecOps.cshtml | 3 + src/webapp01/Pages/DevSecOps2.cshtml | 252 ++++++++++++++++++++++++ src/webapp01/Pages/DevSecOps2.cshtml.cs | 215 ++++++++++++++++++++ src/webapp01/Pages/Index.cshtml | 4 + 4 files changed, 474 insertions(+) create mode 100644 src/webapp01/Pages/DevSecOps2.cshtml create mode 100644 src/webapp01/Pages/DevSecOps2.cshtml.cs diff --git a/src/webapp01/Pages/DevSecOps.cshtml b/src/webapp01/Pages/DevSecOps.cshtml index 19f5d71..468a503 100644 --- a/src/webapp01/Pages/DevSecOps.cshtml +++ b/src/webapp01/Pages/DevSecOps.cshtml @@ -139,6 +139,9 @@ Secret Scanning + + Advanced Demo + diff --git a/src/webapp01/Pages/DevSecOps2.cshtml b/src/webapp01/Pages/DevSecOps2.cshtml new file mode 100644 index 0000000..142ce3a --- /dev/null +++ b/src/webapp01/Pages/DevSecOps2.cshtml @@ -0,0 +1,252 @@ +@page +@model DevSecOps2Model +@{ + ViewData["Title"] = "Advanced DevSecOps Security Demonstrations"; +} + +
+
+
+

@ViewData["Title"]

+

Extended security vulnerability demonstrations for GitHub Advanced Security scanning

+
+
+
+ + + @if (TempData["SqlResult"] != null) + { + + } + + @if (TempData["SqlError"] != null) + { + + } + +
+ +
+
+
+

+ Advanced Security Vulnerabilities Demo +

+
+
+ @if (Model.SecurityDemos.Any()) + { +
+ @foreach (var demo in Model.SecurityDemos) + { +
+ VULN +
+

@demo

+ Detected by GHAS Code Scanning +
+
+ } +
+ } + else + { +

No vulnerability demonstrations available.

+ } +
+
+ + +
+
+

Extended GHAS Capabilities

+
+
+
+
+
Advanced Code Analysis
+

Deep semantic analysis with custom CodeQL queries for complex vulnerability patterns.

+ +
SQL Injection Detection
+

Automated detection of SQL injection vulnerabilities in database queries.

+
+
+
Custom Security Rules
+

Organization-specific security policies and custom vulnerability detection rules.

+ +
Supply Chain Security
+

Comprehensive dependency vulnerability tracking and remediation guidance.

+
+
+
+
+ + +
+
+

Security Metrics Dashboard

+
+
+
+
+

@Model.VulnerabilityCount

+ Critical Vulnerabilities +
+
+

@Model.SecretCount

+ Exposed Secrets +
+
+

@Model.DependencyCount

+ Vulnerable Dependencies +
+
+

@Model.FixedCount

+ Issues Resolved +
+
+
+
+
+ + +
+ +
+
+

+ SQL Injection Demo +

+
+
+

+ This form demonstrates SQL injection vulnerabilities that should be detected by GHAS. + DO NOT use in production! +

+ + +
+
+ + +
+ ⚠️ This query is vulnerable to SQL injection attacks. +
+
+ +
+
+
+ + +
+
+

+ CSRF Demo +

+
+
+

+ This form lacks CSRF protection, demonstrating a common security vulnerability. +

+ + +
+
+ + +
+ +
+
+
+ + +
+
+

Advanced Resources

+
+
+
+ + CodeQL CI Integration + + + Custom Secret Patterns + + + Dependabot Configuration + + + Security Advisories + + + Basic Demo + +
+
+
+
+
+ + +
+
+ +
+
+
+ +@section Scripts { + +} \ No newline at end of file diff --git a/src/webapp01/Pages/DevSecOps2.cshtml.cs b/src/webapp01/Pages/DevSecOps2.cshtml.cs new file mode 100644 index 0000000..8e7d401 --- /dev/null +++ b/src/webapp01/Pages/DevSecOps2.cshtml.cs @@ -0,0 +1,215 @@ +using Microsoft.AspNetCore.Mvc; +using Microsoft.AspNetCore.Mvc.RazorPages; +using Microsoft.Data.SqlClient; +using System.Data; +using System.Security.Cryptography; +using System.Text; + +namespace webapp01.Pages +{ + public class DevSecOps2Model : PageModel + { + private readonly ILogger _logger; + + // Hardcoded API keys and secrets - INSECURE FOR DEMO + private const string API_KEY = "sk-1234567890abcdef1234567890abcdef"; + private const string DATABASE_PASSWORD = "P@ssw0rd123!"; + private const string JWT_SECRET = "MyVerySecretJWTKey123456789"; + + // Insecure connection string with embedded credentials + private const string UNSAFE_CONNECTION_STRING = "Data Source=server.example.com;Initial Catalog=ProductionDB;User ID=sa;Password=SuperSecret123;"; + + public DevSecOps2Model(ILogger logger) + { + _logger = logger; + } + + public List SecurityDemos { get; set; } = new(); + public int VulnerabilityCount { get; set; } + public int SecretCount { get; set; } + public int DependencyCount { get; set; } + public int FixedCount { get; set; } + + public void OnGet() + { + // Log forging vulnerability - direct user input logging + string userAgent = Request.Headers.UserAgent.ToString() ?? "Unknown"; + string clientIP = Request.HttpContext.Connection.RemoteIpAddress?.ToString() ?? "Unknown"; + _logger.LogInformation($"DevSecOps2 page accessed from IP: {clientIP} with User-Agent: {userAgent}"); + + // Load security demonstration data + LoadSecurityDemos(); + LoadSecurityMetrics(); + + // Demonstrate insecure cryptographic practices + DemonstrateWeakCrypto(); + + // Simulate unsafe file operations + SimulateFileOperations(); + } + + private void LoadSecurityDemos() + { + SecurityDemos = new List + { + "SQL Injection vulnerability in user search functionality", + "Cross-Site Request Forgery (CSRF) protection disabled", + "Hardcoded API keys and database credentials in source code", + "Weak cryptographic algorithms (MD5, DES) in use", + "Path traversal vulnerability in file download feature", + "Insecure direct object references in user data access", + "Missing input validation on user-supplied data", + "Sensitive data logged in plain text format", + "Unsafe deserialization of untrusted data", + "Information disclosure through verbose error messages" + }; + + _logger.LogInformation($"Loaded {SecurityDemos.Count} security vulnerability demonstrations"); + } + + private void LoadSecurityMetrics() + { + // Simulated security metrics for demonstration + VulnerabilityCount = 15; + SecretCount = 8; + DependencyCount = 23; + FixedCount = 42; + + // Log sensitive information - INSECURE + _logger.LogWarning($"Security scan results: {VulnerabilityCount} critical issues found with API key: {API_KEY}"); + } + + private void DemonstrateWeakCrypto() + { + try + { + // Use of weak cryptographic algorithm - MD5 + using (var md5 = MD5.Create()) + { + string sensitiveData = "user:admin,password:secret123"; + byte[] hash = md5.ComputeHash(Encoding.UTF8.GetBytes(sensitiveData)); + string hashString = Convert.ToBase64String(hash); + + // Log sensitive hash + _logger.LogInformation($"Generated MD5 hash for sensitive data: {hashString}"); + } + + // Weak random number generation + Random weakRandom = new Random(12345); // Predictable seed + int sessionToken = weakRandom.Next(1000, 9999); + + _logger.LogInformation($"Generated session token: {sessionToken} using weak randomization"); + } + catch (Exception ex) + { + // Information disclosure through detailed error messages + _logger.LogError($"Cryptographic operation failed: {ex.Message} | Stack: {ex.StackTrace}"); + } + } + + private void SimulateFileOperations() + { + try + { + // Path traversal vulnerability simulation + string fileName = Request.Query.ContainsKey("file") ? Request.Query["file"].ToString() ?? "default.txt" : "default.txt"; + string fullPath = Path.Combine("/app/data", fileName); // Unsafe path combination + + _logger.LogInformation($"Attempting to access file: {fullPath}"); + + // Command injection vulnerability (simulated) + string command = $"ls -la {fullPath}"; + _logger.LogInformation($"Executing command: {command}"); + + } + catch (Exception ex) + { + _logger.LogError($"File operation failed: {ex}"); + } + } + + public IActionResult OnPostTestSql(string username) + { + if (string.IsNullOrEmpty(username)) + { + TempData["SqlError"] = "Username cannot be empty"; + return RedirectToPage(); + } + + try + { + // SQL Injection vulnerability - direct string concatenation + string sqlQuery = $"SELECT * FROM Users WHERE Username = '{username}'"; + + // Log the vulnerable SQL query + _logger.LogInformation($"Executing SQL query: {sqlQuery}"); + + // Simulate database connection (don't actually execute) + using var connection = new SqlConnection(UNSAFE_CONNECTION_STRING); + TempData["SqlResult"] = $"Query executed: {sqlQuery}"; + + // Log user input without sanitization + _logger.LogInformation($"User search performed for: {username}"); + } + catch (Exception ex) + { + // Information disclosure in error handling + _logger.LogError($"SQL operation failed for user '{username}': {ex.Message} | Connection: {UNSAFE_CONNECTION_STRING}"); + TempData["SqlError"] = $"Database error: {ex.Message}"; + } + + return RedirectToPage(); + } + + public IActionResult OnPostUnsafeAction(string action) + { + // CSRF vulnerability - no anti-forgery token validation + // Missing authorization checks + + if (string.IsNullOrEmpty(action)) + { + return BadRequest("Action parameter required"); + } + + try + { + // Log forging vulnerability + _logger.LogInformation($"Unsafe action executed: {action} by user from IP: {Request.HttpContext.Connection.RemoteIpAddress}"); + + switch (action.ToLower()) + { + case "delete": + // Simulate dangerous operation without proper authorization + _logger.LogWarning($"Delete operation executed with API key: {API_KEY}"); + TempData["SqlResult"] = "Delete operation simulated (CSRF vulnerable)"; + break; + + case "update": + // Expose sensitive configuration + _logger.LogInformation($"Update operation with database password: {DATABASE_PASSWORD}"); + TempData["SqlResult"] = "Update operation simulated (no authorization)"; + break; + + default: + TempData["SqlResult"] = $"Action '{action}' executed without CSRF protection"; + break; + } + + // Insecure redirect + string returnUrl = Request.Query["returnUrl"].ToString(); + if (!string.IsNullOrEmpty(returnUrl)) + { + return Redirect(returnUrl); // Open redirect vulnerability + } + } + catch (Exception ex) + { + // Detailed error information disclosure + _logger.LogError($"Action '{action}' failed: {ex} | JWT Secret: {JWT_SECRET}"); + TempData["SqlError"] = $"Operation failed: {ex.Message}"; + } + + return RedirectToPage(); + } + } +} \ No newline at end of file diff --git a/src/webapp01/Pages/Index.cshtml b/src/webapp01/Pages/Index.cshtml index e0db7f6..c3d9411 100644 --- a/src/webapp01/Pages/Index.cshtml +++ b/src/webapp01/Pages/Index.cshtml @@ -13,5 +13,9 @@ New! Check out our DevSecOps Demo page to see the latest GHAS features and security demonstrations.

+

+ Extended Demo: Visit our Advanced DevSecOps + page for additional security vulnerability demonstrations and extended GHAS capabilities. +