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 future_dates=ignore_entry scrub #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jeblair
Copy link

@jeblair jeblair commented Jan 15, 2016

The ignore_entry setting for the future_dates scrub incorrectly compared dates
to time.time() (a float) rather than time.gmtime() (a time_struct).

The ignore_entry setting for the future_dates scrub incorrectly compared dates
to time.time() (a float) rather than time.gmtime() (a time_struct).
openstack-gerrit pushed a commit to openstack/openstack-planet that referenced this pull request Jan 18, 2016
If an entry arrives with a date in the future, disregard the date
and use the internal fetch date instead.

Note, the ignore_entry option is not safe to use without this patch:
  rubys/venus#35

Change-Id: If2c6ea31c8d60211e93615c6104d193e1970e271
openstack-gerrit pushed a commit to openstack/openstack that referenced this pull request Jan 18, 2016
Project: openstack/openstack-planet  156b7996ec72cc532d68103b3c3b4bfb75a6c060

Ignore future dates

If an entry arrives with a date in the future, disregard the date
and use the internal fetch date instead.

Note, the ignore_entry option is not safe to use without this patch:
  rubys/venus#35

Change-Id: If2c6ea31c8d60211e93615c6104d193e1970e271
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