Skip to content
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

Sync 1.5.x into main #8320

Closed
wants to merge 117 commits into from
Closed

Sync 1.5.x into main #8320

wants to merge 117 commits into from

Conversation

stnguyen90
Copy link
Contributor

What does this PR do?

(Provide a description of what this PR does and why it's needed.)

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?

lohanidamodar and others added 30 commits April 8, 2024 05:03
… chore-error-requests-count

� Conflicts:
�	app/controllers/shared/api.php
�	composer.json
stnguyen90 and others added 29 commits June 21, 2024 08:46
…rations

Add `default` to Collection Attributes in Migration
There seems to be a load balancer in front of www.google.com that has
either a certificate issuer of "Google Trust Services LLC" or
"Google Trust Services". This causes the test to fail intermittently.

This PR updates the test to check for the substring
"Google Trust Services" instead of the exact string
"Google Trust Services LLC".
The execution is async so give the function a bit more time to execute.
Before this, the Create session API call would throw:

> Invalid document structure: Missing required attribute "expire"

This is because the `expire` attribute is required, but it was omitted
from the document. This PR ensures the `expire` attribute is set when
creating the session document.
The create session endpoint created a 6 character secret which is too
short. This changes the secret to be 256 characters which is in line
with the secret for `account.createEmailPasswordSession()`.
utopia-php/database version 0.50.0 includes a new skipValidation()
function we can use to skip validation when making queries.
If there are more than 100 targets, the validation throws an error. This
change skips the validation so no error is thrown.
Disable validation for subquery to prevent error
…ating-session

Fix 'Missing required attribute "expire"' on `users.createSession()`
Update Expires header via cache shared api
Sync refactor-usage-sn into 1.5.x
@stnguyen90 stnguyen90 closed this Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants