mirror of
https://github.com/empayre/fleet.git
synced 2024-11-06 00:45:19 +00:00
57816592ba
not set on the INSERT.
- OUT: Only sets the ID on the passed session and returns it. (`CreatedAt`, `AccessedAt`, are not set.)
New version:
```go
func (ds *Datastore) NewSession(ctx context.Context, userID uint, sessionKey string) (*fleet.Session, error) {
sqlStatement := `
INSERT INTO sessions (
user_id,
` + "`key`" + `
)
VALUES(?,?)
`
result, err := ds.writer.ExecContext(ctx, sqlStatement, userID, sessionKey)
if err != nil {
return nil, ctxerr.Wrap(ctx, err, "inserting session")
}
id, _ := result.LastInsertId() // cannot fail with the mysql driver
return ds.sessionByID(ctx, ds.writer, uint(id))
}
```
- IN: Define arguments that are truly used when creating a session.
- OUT: Load and return the fleet.Session struct with all values set (using the `ds.writer` to support read replicas correctly).
PS: The new `NewSession` version mimics what we already do with other entities, like policies (`Datastore.NewGlobalPolicy`).
|
||
|---|---|---|
| .. | ||
| api | ||
| app | ||
| backup_db | ||
| bomutils-docker | ||
| ci | ||
| dbutils | ||
| desktop | ||
| docker | ||
| file-server | ||
| fleetctl-npm | ||
| loadtesting/terraform | ||
| mysql-replica-testing | ||
| osquery | ||
| redis-stress | ||
| redis-tests | ||
| saml | ||
| terraform | ||
| tuf | ||
| wix-docker | ||