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

fix: erroneous upload chunk request data consumption #23

Merged

Conversation

kennedykori
Copy link
Contributor

This fixes the issue resulting on error 400 when submitting upload chunks due to erroneous request data consumption.

Fix issue resulting on error 400 when submitting upload chunks due to erroneous request data consumption.
@kennedykori kennedykori marked this pull request as ready for review August 14, 2022 10:18
@kennedykori kennedykori merged commit 6aaddf8 into develop Aug 14, 2022
kennedykori added a commit that referenced this pull request Sep 11, 2022
Fix issue resulting on error 400 when submitting upload chunks due to erroneous request data consumption.
kennedykori added a commit that referenced this pull request Sep 11, 2022
Fix issue resulting on error 400 when submitting upload chunks due to erroneous request data consumption.
github-actions bot pushed a commit that referenced this pull request Sep 11, 2022
## [1.3.0](v1.2.0...v1.3.0) (2022-09-11)

### Features

* Initial implementation for dashboards and visualizations app ([#15](#15)) ([33a6306](33a6306))
* integrate pub/sub to notify new uploads ([#26](#26)) ([d6ae0f1](d6ae0f1))

### Bug Fixes

* **api:** add missing field on upload chunk creation ([#31](#31)) ([4565996](4565996))
* **docker:** Dockerfile ([#24](#24)) ([318ace2](318ace2))
* drf missing static assets ([#25](#25)) ([3b1e745](3b1e745))
* **api:** erroneous upload chunk request data consumption ([#23](#23)) ([759f3a8](759f3a8))
* PubSub message creation ([#36](#36)) ([0f9bf8a](0f9bf8a))
* PubSub message to JSON conversion ([#37](#37)) ([4962190](4962190)), closes [#36](#36)

### Dependency Updates

* **deps:** bump django from 4.0.4 to 4.0.6 ([#16](#16)) ([58e3455](58e3455))
* **deps:** bump jquery-validation from 1.19.4 to 1.19.5 ([#17](#17)) ([6f9aeee](6f9aeee))
* **deps:** bump terser from 5.13.1 to 5.14.2 ([#19](#19)) ([06e7956](06e7956))

### Refactors

* **config:** add pub_sub_topic environment variable ([#34](#34)) ([446cc4a](446cc4a))
* add unit tests ([#22](#22)) ([ade99da](ade99da))
* data_source & metadata_extract filters ([#18](#18)) ([07e6c6b](07e6c6b))
* **conf:** improve project configuration ([#21](#21)) ([f8e1614](f8e1614))
* **pubsub:** refactor pubsub event trigger ([#30](#30)) ([43f1c9a](43f1c9a))
* **core:** refactor upload metadata architecture. ([#28](#28)) ([781ec68](781ec68))
* **api:** remove unneeded fields from the start upload chunk api. ([#27](#27)) ([3a498b6](3a498b6))
@kennedykori kennedykori deleted the fix-chunk-upload-erroneous-request-data-consumption branch September 11, 2022 19:26
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.

1 participant