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

Exception parsing ETag of Azurite response #95

Closed
capoitas opened this issue Sep 7, 2018 · 2 comments
Closed

Exception parsing ETag of Azurite response #95

capoitas opened this issue Sep 7, 2018 · 2 comments
Labels
enhancement New feature or request

Comments

@capoitas
Copy link

capoitas commented Sep 7, 2018

We are using the latest version of the Docker Azurite. We can access it through the Azure Explorer but...

Azurite is returning a 200 response with:
{ETag: W/"75-mAoh3qooM6Mzsi98Dku2rmIXONw"}

As opposed to the same response coming from the real Azure with the format:
{ETag: W/"datetime'2018-09-07T14:42:56.757613Z'"}

Is this a know issue, a new issue or are we doing something wrong?

@XiaoningLiu XiaoningLiu added the enhancement New feature or request label Sep 26, 2018
docschmidt added a commit to docschmidt/Azurite that referenced this issue Jan 29, 2019
* Disable express' etag generation
* Update entity operations to emit etag according to spec
* Update `EntityIfMatch` to match timestamp instead of etag
* Add unit tests
@GSingha
Copy link

GSingha commented Apr 5, 2019

I am facing the same issue. When the change will be released?

edwin-huber added a commit that referenced this issue Aug 28, 2019
* sped up tests by removing polling

* Fix query single item incorrect behavior

* Fixes incorrect handling of table management URLs

* Ensures only one AzuriteTableRequest is created and processed

* Updated paths so tests don't rely on cloning directory (#105)

Re-targeted to dev branch for pull request...

* Made etag handling same as in microsoft

* Return value collection with only one item when query type is not a single entity query

* Unit test

* Unit test

* Make "initialized" a member of the Environment class

Although the "env" module behaves as a singleton,
the "initialized" variable should be within the object's
definition. This allows better reusability and testing.

* Add tests for Environment class

* Allow 0 to be a valid port number

Passing port number 0 to Node.js' server.listen() will make it assign
an arbitrary unused port.

* Fixes #95

* Disable express' etag generation
* Update entity operations to emit etag according to spec
* Update `EntityIfMatch` to match timestamp instead of etag
* Add unit tests

* merge for #148 poveden:poveden/proper-close

* Fixed merge issues

* removed duplicated Basic_blob_tests

* Method override support

* Support case when RowKey is empty string

* added test and corrected status code

* Add support for HTTP Method Overrides (#128)

* using correct process termination message for cleanup on exit in windows, also applied NPM audit fixes

* merged changes

* Updated release notes for 2.7.1 release

* updated package dependencies with npm audit fix, all tests still passing

* Updated package version

* removed extra -
edwin-huber added a commit that referenced this issue Feb 4, 2020
* sped up tests by removing polling

* Fix query single item incorrect behavior

* Fixes incorrect handling of table management URLs

* Ensures only one AzuriteTableRequest is created and processed

* Updated paths so tests don't rely on cloning directory (#105)

Re-targeted to dev branch for pull request...

* Made etag handling same as in microsoft

* Return value collection with only one item when query type is not a single entity query

* Unit test

* Unit test

* Make "initialized" a member of the Environment class

Although the "env" module behaves as a singleton,
the "initialized" variable should be within the object's
definition. This allows better reusability and testing.

* Add tests for Environment class

* Allow 0 to be a valid port number

Passing port number 0 to Node.js' server.listen() will make it assign
an arbitrary unused port.

* Fixes #95

* Disable express' etag generation
* Update entity operations to emit etag according to spec
* Update `EntityIfMatch` to match timestamp instead of etag
* Add unit tests

* merge for #148 poveden:poveden/proper-close

* Fixed merge issues

* removed duplicated Basic_blob_tests

* Method override support

* Support case when RowKey is empty string

* added test and corrected status code

* Add support for HTTP Method Overrides (#128)

* using correct process termination message for cleanup on exit in windows, also applied NPM audit fixes

* merged changes

* Escaped quotation marks replacement fix. (#284)

Was also replacing all @ by single quotes, breaking matches with strings containing @ (i.e. email addresses).

* updated release notes

* minor formatting of release note

* updated version in package.json

Co-authored-by: Alex <[email protected]>
Co-authored-by: Phred Lane <[email protected]>
Co-authored-by: Kyle Dodson <[email protected]>
Co-authored-by: Michael Kruglos <3526988 [email protected]>
Co-authored-by: Patryk Grzegorczyk <[email protected]>
Co-authored-by: Jorge Poveda <[email protected]>
Co-authored-by: Alexander Schmidt <[email protected]>
Co-authored-by: Patryk Grzegorczyk <[email protected]>
Co-authored-by: Richard Cognot <[email protected]>
edwin-huber added a commit that referenced this issue Apr 29, 2020
* sped up tests by removing polling

* Fix query single item incorrect behavior

* Fixes incorrect handling of table management URLs

* Ensures only one AzuriteTableRequest is created and processed

* Updated paths so tests don't rely on cloning directory (#105)

Re-targeted to dev branch for pull request...

* Made etag handling same as in microsoft

* Return value collection with only one item when query type is not a single entity query

* Unit test

* Unit test

* Make "initialized" a member of the Environment class

Although the "env" module behaves as a singleton,
the "initialized" variable should be within the object's
definition. This allows better reusability and testing.

* Add tests for Environment class

* Allow 0 to be a valid port number

Passing port number 0 to Node.js' server.listen() will make it assign
an arbitrary unused port.

* Fixes #95

* Disable express' etag generation
* Update entity operations to emit etag according to spec
* Update `EntityIfMatch` to match timestamp instead of etag
* Add unit tests

* merge for #148 poveden:poveden/proper-close

* Fixed merge issues

* removed duplicated Basic_blob_tests

* Method override support

* Support case when RowKey is empty string

* added test and corrected status code

* Add support for HTTP Method Overrides (#128)

* using correct process termination message for cleanup on exit in windows, also applied NPM audit fixes

* merged changes

* Escaped quotation marks replacement fix. (#284)

Was also replacing all @ by single quotes, breaking matches with strings containing @ (i.e. email addresses).

* updated release notes

* minor formatting of release note

* updated version in package.json

* updated legacy dev readme to remove references to old arafato docker hub entry and point to v3 for all needs other than table storage (#393)

Co-authored-by: Alex <[email protected]>
Co-authored-by: Phred <[email protected]>
Co-authored-by: Alex Swan <[email protected]>
Co-authored-by: Kyle Dodson <[email protected]>
Co-authored-by: Michael Kruglos <[email protected]>
Co-authored-by: Patryk Grzegorczyk (CodeTwo) <[email protected]>
Co-authored-by: Jorge Poveda <[email protected]>
Co-authored-by: Alexander Schmidt <[email protected]>
Co-authored-by: Edwin Huber <[email protected]>
Co-authored-by: Patryk Grzegorczyk <[email protected]>
Co-authored-by: Richard Cognot <[email protected]>
@edwin-huber
Copy link
Collaborator

eTag checking and formatting now in v3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants