8000 Add new console endpoint to check resource availability by vermakhushboo · Pull Request #8970 · appwrite/appwrite · GitHub

More Web Proxy on the site http://driver.im/

Add new console endpoint to check resource availability #8970

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.

Sign up for GitHub

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

Jump to bottom

Merged

Meldiron merged 21 commits into feat-sites from feat-console-availability-endpoint

Feb 12, 2025

Member

vermakhushboo commented

What does this PR do?

Add new console endpoint to check for resource availability like subdomain etc.

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)

Related PRs and Issues

(Related PR or issue)

Checklist

Have you read the Contributing Guidelines on issues?
If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?


          WIP: Add new console endpoint

d95cdc3

vermakhushboo requested review from eldadfux and TorstenDittmann

November 13, 2024 16:17

vermakhushboo marked this pull request as draft

November 13, 2024 16:17

github-actions bot commented

•

✨ Benchmark results

Requests per second: 851
Requests with 200 status code: 153,146
P99 latency: 0.226389669

⚡ Benchmark Comparison

Metric	This PR	Latest version
RPS	851	1,592
200	153,146	286,573
P99	0.226389669	0.072041888

vermakhushboo marked this pull request as ready for review

November 13, 2024 18:24

eldadfux requested changes

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved

app/controllers/api/console.php Outdated Show resolved Hide resolved

app/controllers/api/console.php Outdated Show resolved Hide resolved

src/Appwrite/Platform/Modules/Sites/Http/Sites/CreateSite.php Outdated Show resolved Hide resolved

src/Appwrite/Platform/Modules/Sites/Http/Sites/CreateSite.php Outdated Show resolved Hide resolved

app/controllers/api/console.php Outdated Show resolved Hide resolved

app/controllers/api/console.php Outdated Show resolved Hide resolved

eldadfux requested changes

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved


          Use resourceId in the endpoint url

7836e48

github-actions bot commented

•

Security Scan Results for PR

Docker Image Scan Results

Package	Version	Vulnerability	Severity
libcrypto3	3.3.2-r1	CVE-2024-12797	HIGH
libssl3	3.3.2-r1	CVE-2024-12797	HIGH
openssl	3.3.2-r1	CVE-2024-12797	HIGH
openssl-dev	3.3.2-r1	CVE-2024-12797	HIGH
pyc	3.12.7-r0	CVE-2024-12254	HIGH
pyc	3.12.7-r0	CVE-2024-9287	HIGH
python3	3.12.7-r0	CVE-2024-12254	HIGH
python3	3.12.7-r0	CVE-2024-9287	HIGH
python3-pyc	3.12.7-r0	CVE-2024-12254	HIGH
python3-pyc	3.12.7-r0	CVE-2024-9287	HIGH
python3-pycache-pyc0	3.12.7-r0	CVE-2024-12254	HIGH
python3-pycache-pyc0	3.12.7-r0	CVE-2024-9287	HIGH
rsync	3.3.0-r0	CVE-2024-12084	CRITICAL
rsync	3.3.0-r0	CVE-2024-12085	HIGH
golang.org/x/crypto	v0.26.0	CVE-2024-45337	CRITICAL
golang.org/x/net	v0.28.0	CVE-2024-45338	HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

Meldiron reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved

vermakhushboo added 3 commits

January 13, 2025 00:10


          Resolve merge conflict

652ab49


          Remove desc as private endpoint

dd26a56


          Add abuse limit

aa13007

vermakhushboo changed the title ~~WIP: Add new console endpoint~~ Add new console endpoint to check resource availability

vermakhushboo requested review from Meldiron and eldadfux

January 12, 2025 19:54

christyjacob4 reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved


          Remove redundant check

76a0167

christyjacob4 reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved

christyjacob4 reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved

christyjacob4 reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved

christyjacob4 reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved


          Remove unnecessary check

246dc07

christyjacob4 approved these changes

View reviewed changes


          Resolve merge conflicts

0124d52

Meldiron reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved


          Add rules.read scope to console member

7be0f4a

Meldiron reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

app/controllers/api/console.php Outdated Show resolved Hide resolved

vermakhushboo added 5 commits

February 7, 2025 16:32


          Resolve merge conflicts

eaa4795


          Add tests

27de266


          Update SDK method structure

0dc4343


          Rename to getResourceAvailability

< 6D4E batch-deferred-content class="d-inline-block" data-url="/appwrite/appwrite/commits/checks-statuses-rollups">

a322b02


          Change to module structure

681f4b9

Meldiron reviewed

View reviewed changes

tests/e2e/Services/Sites/SitesCustomServerTest.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

tests/e2e/Services/Sites/SitesCustomServerTest.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

tests/e2e/Services/Sites/SitesCustomServerTest.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

src/Appwrite/Platform/Modules/Console/Http/Resources/Get.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

src/Appwrite/Platform/Modules/Console/Http/Resources/Get.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

src/Appwrite/Platform/Modules/Console/Http/Resources/Get.php Outdated Show resolved Hide resolved

vermakhushboo added 2 commits

February 12, 2025 11:49


          Resolve merge conflicts

b46eebb


          Refactor console availability endpoint

aae3b40

vermakhushboo requested a review from Meldiron

February 12, 2025 10:42

Meldiron reviewed

View reviewed changes

src/Appwrite/Platform/Modules/Console/Http/Resources/Get.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

src/Appwrite/Platform/Modules/Console/Http/Resources/Get.php Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

tests/e2e/Services/Sites/SitesCustomServerTest.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

tests/e2e/Services/Sites/SitesCustomServerTest.php Outdated Show resolved Hide resolved

Meldiron reviewed

View reviewed changes

tests/e2e/Services/Sites/SitesCustomServerTest.php Outdated Show resolved Hide resolved


          Use getSiteDomain

adb861b

vermakhushboo requested a review from Meldiron

February 12, 2025 12:02

vermakhushboo added 3 commits

February 12, 2025 17:54


          Increase time to assert

c78701d


          Increase deployment build timeout

3b0d760


          Update Get.php

0106dc7

Meldiron reviewed

View reviewed changes

tests/e2e/Services/Sites/SitesBase.php Show resolved Hide resolved

Meldiron approved these changes

View reviewed changes

Meldiron merged commit c414256 into feat-sites

64 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

0