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

Oplog skipped even if config.OPLOG=True #1095

Closed
wants to merge 2 commits into from
Closed

Conversation

lexhung
Copy link
Contributor

@lexhung lexhung commented Dec 24, 2017

Check for resource name validity is being reversed.

Check for resource name validity is being reversed.
@nicolaiarocci
Copy link
Member

Hello, is this supposed to fix #1074? Thanks!

@lexhung lexhung closed this Jan 4, 2018
@lexhung lexhung reopened this Jan 4, 2018
@lexhung
Copy link
Contributor Author

lexhung commented Jan 4, 2018

@nicolaiarocci Yes, it is supposed to fix that.

@nicolaiarocci nicolaiarocci added this to the 0.8 milestone Jan 8, 2018
@nicolaiarocci
Copy link
Member

Would be great if you could add a regression test here.

nicolaiarocci added a commit that referenced this pull request Mar 30, 2018
The original test was a false positive.
nicolaiarocci added a commit that referenced this pull request Mar 30, 2018
@nicolaiarocci
Copy link
Member

nicolaiarocci commented Mar 30, 2018

Added a regression test (or I should say: fixed a false positive test), rebased and merged. Thanks.

3996d90

lexhung added a commit to lexhung/eve that referenced this pull request Dec 12, 2018
* upstream/master: (32 commits)
  Get rid of unwanted .vscode folder
  Add support for mongo $box geo query operator
  Marsch Huynh
  Improve partial downloads documentation
  Changelog for pyeve#1050
  Test coverage for pyeve#1050
  A little refactoring (DRY). Addresses pyeve#1050.
  fix: media endpoint
  Support partial request for media resource
  Officially deprecate Python 2.6
  Changelog for pyeve#1130
  Reduced error description details
  Fix PUT behavior with User-Restricted Resource Access
  flake8
  typo
  Hung Le
  Changelog for pyeve#1095.
  Regression test for PR pyeve#1095.
  Oplog skipped even if confg.OPLOG=True
  DHuan
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants