search

saguziel / Kryptose

A password manager
2 stars 0 forks source link

Logs #14

Open saguziel opened 9 years ago

saguziel commented 9 years ago

So, should we have a logify method in the requests/ responses that turns it into something we can write to the log.

jnshi commented 9 years ago

True that. On Mar 15, 2015 11:05 PM, "saguziel" notifications@github.com wrote:

So, should we have a logify method in the requests/ responses that turns it into something we can write to the log.

— Reply to this email directly or view it on GitHub https://github.com/saguziel/Kryptose/issues/14.

jnshi commented 9 years ago

It'd be good to use the http://en.wikipedia.org/wiki/Syslog format as discussed in lecture. http://www.cs.cornell.edu/courses/CS5430/2015sp/notes/audit.php

AMarcedone commented 9 years ago

We should probably log SSL connections other than just simple formatted requests, and record information like the ip address...

jnshi commented 9 years ago

True. We might also want to use this package of utilities: http://docs.oracle.com/javase/7/docs/api/java/util/logging/package-summary.html

(BTW, why does the current Log implementation use a LocalDateTime? I would think that having an absolute time is a good thing (as opposed to a time relative to an undefined time zone).)

jnshi commented 9 years ago

I'm implementing logging using this class: http://docs.oracle.com/javase/7/docs/api/java/util/logging/Logger.html

AMarcedone commented 9 years ago

Should we consider encrypting the logs for the beta? Perhaps using the protocol described in the lecture notes?

jefftn commented 9 years ago

I think we should do this, and I think we also told them that we intended to do this by beta.