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.

Sign up for GitHub

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

Jump to bottom

assh config list shows odd value for ServerAliveInterval #181

Closed
ocean90 opened this issue Nov 9, 2016 · 3 comments
Closed

assh config list shows odd value for ServerAliveInterval #181

ocean90 opened this issue Nov 9, 2016 · 3 comments
Assignees
@moul
Milestone
v2.6.0

Comments

@ocean90
Copy link
Contributor

ocean90 commented Nov 9, 2016 

➜  ~ assh -v
assh version 2.4.1+dev (HEAD)

➜  ~ assh config build | grep ServerAliveInterval
  ServerAliveInterval 60

➜  ~ assh config list | grep ServerAliveInterval
        ServerAliveInterval: <

Any idea where the < is coming from?
@moul
Copy link
Owner

moul commented Nov 9, 2016

Ahah, fun issue,

“<“ is the ascii char for the decimal 60 value

I will fix it, thanks for the report

On 9 Nov 2016, at 16:59, Dominik Schilling notifications@github.com wrote:

➜ ~ assh -v
assh version 2.4.1+dev (HEAD)

➜ ~ assh config build | grep ServerAliveInterval
ServerAliveInterval 60

➜ ~ assh config list | grep ServerAliveInterval
ServerAliveInterval: <
Any idea where the < is coming from?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub #181, or mute the thread https://github.com/notifications/unsubscribe-auth/AAFvTU9z1PBe-G023Bbw2WHU-X6YZXk7ks5q8e33gaJpZM4Ktr4g.

@moul moul self-assigned this Dec 4, 2016
@moul moul added the ready label Dec 6, 2016
@moul moul modified the milestone: v2.6.0 Jan 5, 2017
@moul moul mentioned this issue Jan 11, 2017
Open
moul added a commit that referenced this issue Jan 11, 2017
@moul
Fix integers output in assh config list (fix #181)
7d352c5 
Signed-off-by: Manfred Touron <m@42.am>
@moul moul mentioned this issue Jan 11, 2017
Merged
@moul moul added in progress and removed ready labels Jan 11, 2017
@moul moul closed this as completed in #193 Jan 11, 2017
@moul moul removed the in progress label Jan 11, 2017
@moul
Copy link
Owner

moul commented Jan 11, 2017

Fixed by #193, can you check again?

@ocean90
Copy link
Contributor Author

ocean90 commented Jan 11, 2017

@moul Works like a charm, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@moul moul

Labels
None yet
Projects
None yet
Milestone
v2.6.0
Development

No branches or pull requests
2 participants
@moul @ocean90