Skip to content

Conversation

justinlettau
Copy link
Contributor

New Pull Request Checklist

Issue Description

Related issue: #1510

Approach

Adding a failing unit test for reproducing base64 regex issue.

TODOs before merging

  • Add tests

@parse-github-assistant
Copy link

parse-github-assistant bot commented Jul 27, 2022

Thanks for opening this pull request!

  • 🎉 We are excited about your hands-on contribution!

@justinlettau justinlettau changed the title chore: failing unit test for #1510 test: failing unit test for #1510 Jul 27, 2022
@mtrezza mtrezza changed the title test: failing unit test for #1510 fix: creating a Parse.File with base64 string fails Jul 28, 2022
@mtrezza mtrezza linked an issue Jul 28, 2022 that may be closed by this pull request
4 tasks
@codecov
Copy link

codecov bot commented Jul 28, 2022

Codecov Report

Merging #1517 (f36f353) into alpha (31cf433) will not change coverage.
The diff coverage is n/a.

❗ Current head f36f353 differs from pull request most recent head ddd8b6c. Consider uploading reports for the commit ddd8b6c to get more accurate results

@@           Coverage Diff           @@
##            alpha    #1517   +/-   ##
=======================================
  Coverage   99.94%   99.94%           
=======================================
  Files          61       61           
  Lines        5955     5955           
  Branches     1358     1358           
=======================================
  Hits         5952     5952           
  Misses          3        3           
Impacted Files Coverage Δ
src/ParseFile.js 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fc79281...ddd8b6c. Read the comment docs.

Copy link
Member

@mtrezza mtrezza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@mtrezza
Copy link
Member

mtrezza commented Jul 28, 2022

off-topic: @dblythy it's interesting that the CI passes here, compared to #1515

@mtrezza mtrezza changed the title fix: creating a Parse.File with base64 string fails fix: creating a Parse.File with base64 string fails for some base64 encodings Jul 28, 2022
@mtrezza mtrezza changed the title fix: creating a Parse.File with base64 string fails for some base64 encodings fix: creating a Parse.File with base64 string fails for some encodings Jul 28, 2022
@mtrezza mtrezza merged commit 0439862 into parse-community:alpha Jul 28, 2022
parseplatformorg pushed a commit that referenced this pull request Jul 28, 2022
## [3.4.4-alpha.1](3.4.3...3.4.4-alpha.1) (2022-07-28)

### Bug Fixes

* creating a Parse.File with base64 string fails for some encodings ([#1517](#1517)) ([0439862](0439862))
* subscription to a LiveQuery containing `ParseQuery.select` overrides properties ([#1488](#1488)) ([b80eee4](b80eee4))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 3.4.4-alpha.1

@parseplatformorg parseplatformorg added the state:released-alpha Released as alpha version label Jul 28, 2022
parseplatformorg pushed a commit that referenced this pull request Oct 25, 2022
# [3.5.0-beta.1](3.4.4...3.5.0-beta.1) (2022-10-25)

### Bug Fixes

* `Parse.User.signUp()` does not pass context to Cloud Code  ([#1527](#1527)) ([53edcfd](53edcfd))
* `Schema.addField` does not correctly add value of type `Date` ([#1544](#1544)) ([15111f7](15111f7))
* creating a Parse.File with base64 string fails for some encodings ([#1517](#1517)) ([0439862](0439862))
* initialization fails in non-browser environment that doesn't support `indexedDB` ([#1569](#1569)) ([3560a5e](3560a5e))
* remove base64 validation due to validation inefficiency ([#1543](#1543)) ([473949d](473949d))

### Features

* add `json` option to `Parse.Query.each()` ([#1539](#1539)) ([89fd5ec](89fd5ec))
* add json option to query.each ([299fb0d](299fb0d))
* generate `Parse.Object.objectId` automatically when `allowCustomObjectId` is enabled and no `objectId` is passed ([#1540](#1540)) ([68f3ff5](68f3ff5))
* localDatastore support for unsorted distance queries ([#1570](#1570)) ([ea3e75f](ea3e75f))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 3.5.0-beta.1

@parseplatformorg parseplatformorg added the state:released-beta Released as beta version label Oct 25, 2022
parseplatformorg pushed a commit that referenced this pull request Nov 1, 2022
# [3.5.0](3.4.4...3.5.0) (2022-11-01)

### Bug Fixes

* `Parse.User.signUp()` does not pass context to Cloud Code  ([#1527](#1527)) ([53edcfd](53edcfd))
* `Schema.addField` does not correctly add value of type `Date` ([#1544](#1544)) ([15111f7](15111f7))
* creating a Parse.File with base64 string fails for some encodings ([#1517](#1517)) ([0439862](0439862))
* initialization fails in non-browser environment that doesn't support `indexedDB` ([#1569](#1569)) ([3560a5e](3560a5e))
* remove base64 validation due to validation inefficiency ([#1543](#1543)) ([473949d](473949d))

### Features

* add `json` option to `Parse.Query.each()` ([#1539](#1539)) ([89fd5ec](89fd5ec))
* add json option to query.each ([299fb0d](299fb0d))
* generate `Parse.Object.objectId` automatically when `allowCustomObjectId` is enabled and no `objectId` is passed ([#1540](#1540)) ([68f3ff5](68f3ff5))
* localDatastore support for unsorted distance queries ([#1570](#1570)) ([ea3e75f](ea3e75f))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 3.5.0

@parseplatformorg parseplatformorg added the state:released Released as stable version label Nov 1, 2022
mtrezza pushed a commit that referenced this pull request Nov 3, 2022
* `Parse.User.signUp()` does not pass context to Cloud Code  ([#1527](#1527)) ([53edcfd](53edcfd))
* `Schema.addField` does not correctly add value of type `Date` ([#1544](#1544)) ([15111f7](15111f7))
* creating a Parse.File with base64 string fails for some encodings ([#1517](#1517)) ([0439862](0439862))
* initialization fails in non-browser environment that doesn't support `indexedDB` ([#1569](#1569)) ([3560a5e](3560a5e))
* remove base64 validation due to validation inefficiency ([#1543](#1543)) ([473949d](473949d))

* add `json` option to `Parse.Query.each()` ([#1539](#1539)) ([89fd5ec](89fd5ec))
* add json option to query.each ([299fb0d](299fb0d))
* generate `Parse.Object.objectId` automatically when `allowCustomObjectId` is enabled and no `objectId` is passed ([#1540](#1540)) ([68f3ff5](68f3ff5))
* localDatastore support for unsorted distance queries ([#1570](#1570)) ([ea3e75f](ea3e75f))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state:released Released as stable version state:released-alpha Released as alpha version state:released-beta Released as beta version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Creating a Parse.File with base64 string fails
3 participants