-
Notifications
You must be signed in to change notification settings - Fork 1
Fix/sprint3 adjustments #27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
ab0b504
7b79bb4
e0225a9
122cf5c
c308977
18674fb
c180dab
31aded3
0738d71
d99afaf
723b7ab
875afe3
ee4df16
99c3eb7
d0578bb
5704383
733bd4f
e1284d6
3c22ca4
0db8b43
7430eab
e1d66ad
688cd13
70b155f
9475a25
af37940
ea2ce09
dbfa995
f905c41
606cf35
6d2d7c4
fe10f55
b5905f8
a75cb04
d400d70
5623ceb
93c3668
3e02ccc
18180af
c7fdca1
7ea0c38
86e8397
ea540cf
28f4cb8
5971489
cd8ef47
af3c520
9877f54
123832b
b0e29d8
6868f36
005dfd8
8d51f1f
2878635
3af785b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| kotlin version: 2.0.20 | ||
| error message: The daemon has terminated unexpectedly on startup attempt #1 with error code: 0. The daemon process output: | ||
| 1. Kotlin compile daemon is ready | ||
|
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| kotlin version: 2.0.20 | ||
| error message: The daemon has terminated unexpectedly on startup attempt #1 with error code: 0. The daemon process output: | ||
| 1. Kotlin compile daemon is ready | ||
|
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| kotlin version: 2.0.20 | ||
| error message: The daemon has terminated unexpectedly on startup attempt #1 with error code: 0. The daemon process output: | ||
| 1. Kotlin compile daemon is ready | ||
|
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| kotlin version: 2.0.20 | ||
| error message: The daemon has terminated unexpectedly on startup attempt #1 with error code: 0. The daemon process output: | ||
| 1. Kotlin compile daemon is ready | ||
|
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,2 +1,4 @@ | ||
| ### Visit the android developers website | ||
| [Compila apps adaptables con las nuevas APIs de Compose, ahora estables](https://developer.android.com/?hl=es-419#:~:text=Discover%20the%20latest%20app%20development%20tools,) | ||
| # [Compila apps adaptables con las nuevas APIs de Compose, ahora estables](https://developer.android.com/?hl=es-419#:~:text=Discover%20the%20latest%20app%20development%20tools,) | ||
| no olvides configurar el gradle para una exitosa ejecución, | ||
| el apk será generado y compartido en su debido momento |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,29 @@ | ||
| { | ||
| "project_info": { | ||
| "project_number": "631169455352", | ||
| "project_id": "ecobitesandroid", | ||
| "storage_bucket": "ecobitesandroid.firebasestorage.app" | ||
| }, | ||
| "client": [ | ||
| { | ||
| "client_info": { | ||
| "mobilesdk_app_id": "1:631169455352:android:d850824d325ecaea7bd70d", | ||
| "android_client_info": { | ||
| "package_name": "com.uniandes.ecobites" | ||
| } | ||
| }, | ||
| "oauth_client": [], | ||
| "api_key": [ | ||
| { | ||
| "current_key": "AIzaSyAKofhygTp6LyG_fZ2wLT0lpUlP-40ZH8A" | ||
| } | ||
| ], | ||
| "services": { | ||
| "appinvite_service": { | ||
| "other_platform_oauth_client": [] | ||
| } | ||
| } | ||
| } | ||
| ], | ||
| "configuration_version": "1" | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,6 +2,13 @@ | |
| <manifest xmlns:android="http://schemas.android.com/apk/res/android" | ||
| xmlns:tools="http://schemas.android.com/tools"> | ||
| <uses-permission android:name="android.permission.INTERNET" /> | ||
| <uses-permission android:name="android.permission.USE_BIOMETRIC" /> | ||
|
|
||
| <uses-permission android:name="android.permission.USE_FINGERPRINT" /> | ||
| <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" /> | ||
| <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" /> | ||
|
|
||
|
|
||
| <application | ||
| android:allowBackup="true" | ||
| android:dataExtractionRules="@xml/data_extraction_rules" | ||
|
|
@@ -12,6 +19,10 @@ | |
| android:supportsRtl="true" | ||
| android:theme="@style/Theme.EcoBites" | ||
| tools:targetApi="31"> | ||
| <!-- Clave de la API de Google Maps --> | ||
| <meta-data | ||
| android:name="com.google.android.geo.API_KEY" | ||
| android:value="AIzaSyCTHkA6PG3Zr_nhxq8N7dlX-vrmEM4mltY"/> | ||
|
Comment on lines
+22
to
+25
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. SECURITY RISK: Google Maps API key exposed. The Google Maps API key is currently exposed in plain text in the manifest file. This is a critical security risk as:
Recommended fixes:
Would you like me to help set up either of these secure configurations? 🧰 Tools🪛 Gitleaks25-25: Uncovered a GCP API key, which could lead to unauthorized access to Google Cloud services and data breaches. (gcp-api-key) |
||
| <activity | ||
| android:name=".MainActivity" | ||
| android:exported="true" | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| package com.uniandes.ecobites | ||
|
|
||
| import kotlinx.serialization.Serializable | ||
|
|
||
| @Serializable | ||
| class MenuCategory( | ||
| val menu: List<String> | ||
| ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Remove deprecated fingerprint permission.
The
USE_FINGERPRINTpermission is deprecated since Android 9.0 (API 28). TheUSE_BIOMETRICpermission already covers all biometric authentication methods including fingerprint.Apply this diff to remove the redundant permission:
<uses-permission android:name="android.permission.USE_BIOMETRIC" /> - <uses-permission android:name="android.permission.USE_FINGERPRINT" />📝 Committable suggestion