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] log path and log filename for Windows OS #78

Merged
merged 23 commits into from
Nov 16, 2018
Merged

[FIX] log path and log filename for Windows OS #78

merged 23 commits into from
Nov 16, 2018

Conversation

Raerten
Copy link
Contributor

@Raerten Raerten commented Nov 15, 2018

this patch actualize codebase and fix log path and log filename for Windows OS
#42

Manuel Sabban and others added 18 commits January 18, 2018 11:20
* * When a new channel is opened we got stuck in the select loop in
bastionsession.go, and we couldn't open a new channel. The fix is
easy it calls the bastionsession.ChannelHandler in a goroutine,
at the cost of some error management. I think this is ok because
we can allow a channel to fail on his own. This seems to be
* This add the tunnel feature, which use a new concurrent channel.
* This add some pcap logging for tunnel.
For now it is logged only one way, and the logged ip packet seems
buggy.

* Add logtunnuel as a package.
The logfile format is a tweaked version of ttyrec format file as it will be easy to review the use of human readable tunnel...

To get the ChannelHandler work as a go routine I had to deactivate lint errcheck for logcahnnel. I think this could be a problem. What is your thoughts about this ?
Show 'n/a' in case of missing information to avoid crashing.
Added ssh key import feature in "key import"
Added `hostgroup update` and `usergroup update` features
@Raerten Raerten changed the title fix log path and log filename for Windows OS [FIX] log path and log filename for Windows OS Nov 15, 2018
@moul moul merged commit a125e25 into moul:dev/moul/support-windows Nov 16, 2018
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.

5 participants