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

[Story/TP-57214]:Prevent reuse of Password reset token #9

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

emborden
Copy link

@emborden emborden commented Jan 6, 2020

Topic

TP Link: https://sbf.tpondemand.com/entity/57214
SBF PR: firespring/SBF#1598

How to test

create a log in api:console create a log with filters and password to a hash notice that the hash will filter the content that is passed in the string

Setup

  • rake switch BRANCH=story/TP-57214

Functionality

  • added the ability to add filters based on log instances.

@emborden emborden changed the title [Story/tp 57214]:Prevent reuse of Password reset token [Story/TP-57214]:Prevent reuse of Password reset token Jan 6, 2020
@@ -1,5 +1,5 @@
# frozen_string_literal: true

class Hedgelog
VERSION = '0.1.12'
VERSION = '0.2.1.alpha1'
Copy link
Contributor

Choose a reason for hiding this comment

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

For future reference standard naming would be to reset the lower significance number when publishing. Since this is a minor version bump, the patch version should be reset, i.e. 0.2.0.alpha.1

https://semver.org/#spec-item-7

Copy link
Author

@emborden emborden Jan 9, 2020

Choose a reason for hiding this comment

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

it was meant to be 0.2.0.alpha1, but when I published I published as just 0.2.0 on accident. So I bumped to 0.2.1.alpha1. to show I was in beta testing, and because of so that minor changes I made to pass spec and rubocop

spec/spec_helper.rb Outdated Show resolved Hide resolved
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