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

RFC-3164 dates have leading zero before date when date has only one digit #11

Merged
merged 1 commit into from
Jul 1, 2013
Merged

Conversation

halimath
Copy link

@halimath halimath commented Jul 1, 2013

Hi,

when using glossy to produce RFC-3164 messages and the day of month only has one digit, the resulting timestamp did not have a leading space before the date (as defined in the RFC). This was due to the replace whitespace statement.

I fixed this issue by filtering and trimming the message parts prior to joining them. I also added a test case to test/produce.js.

Best,
Alex

squeeks added a commit that referenced this pull request Jul 1, 2013
RFC-3164 dates have leading zero before date when date has only one digit
@squeeks squeeks merged commit a714c22 into squeeks:master Jul 1, 2013
@squeeks
Copy link
Owner

squeeks commented Jul 1, 2013

Thanks for the fix, I'll do a release to npm later.

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.

None yet

2 participants