Skip to content

Commit

Permalink
minor whitespace refactorings
Browse files Browse the repository at this point in the history
  • Loading branch information
CommanderStorm committed Sep 13, 2023
1 parent 431e69a commit 3533342
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 24 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ type Token struct {

func NewToken() (*Token, error) {
encryptionKey, err := APNsEncryptionKeyFromFile()

if err != nil {
return nil, err
}
Expand All @@ -50,9 +49,7 @@ func NewToken() (*Token, error) {
TeamId: ApnsTeamId,
}

_, err = token.Generate()

if err != nil {
if err = token.Generate(); err != nil {
return nil, err
}

Expand Down Expand Up @@ -106,7 +103,7 @@ func (t *Token) GenerateNewTokenIfExpired() (bearer string) {
defer t.Unlock()

if t.IsExpired() {
_, err := t.Generate()
err := t.Generate()
if err != nil {
return ""
}
Expand All @@ -119,9 +116,9 @@ func (t *Token) IsExpired() bool {
return currentTimestamp() >= (t.IssuedAt + TokenTimeout)
}

func (t *Token) Generate() (bool, error) {
func (t *Token) Generate() error {
if t.EncryptionKey == nil {
return false, ErrorAuthKeyNil
return ErrorAuthKeyNil
}

issuedAt := currentTimestamp()
Expand All @@ -141,13 +138,13 @@ func (t *Token) Generate() (bool, error) {
token, err := jwtToken.SignedString(t.EncryptionKey)

if err != nil {
return false, err
return err
}

t.IssuedAt = issuedAt
t.Bearer = token

return true, nil
return nil
}

func currentTimestamp() int64 {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -97,19 +97,16 @@ func SymmetricEncrypt(plaintext string, key string) (*EncryptedString, error) {
func SymmetricDecrypt(encryptedString EncryptedString, key string) (string, error) {
bytesKey := []byte(key)
bytesEncryptedString, err := base64.StdEncoding.DecodeString(string(encryptedString))

if err != nil {
return "", err
}

c, err := aes.NewCipher(bytesKey)

if err != nil {
return "", err
}

gcm, err := cipher.NewGCM(c)

if err != nil {
return "", err
}
Expand All @@ -118,7 +115,6 @@ func SymmetricDecrypt(encryptedString EncryptedString, key string) (string, erro
nonce, ciphertext := bytesEncryptedString[:nonceSize], bytesEncryptedString[nonceSize:]

plaintext, err := gcm.Open(nil, nonce, ciphertext, nil)

if err != nil {
return "", err
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -177,13 +177,11 @@ func (service *Service) encryptGradesAndStoreInDatabase(grades []model.IOSGrade,
}

err := encryptedGrade.Encrypt(campusToken)

if err != nil {
log.WithError(err).Error("Could not encrypt grade")
}

err = service.Repository.SaveEncryptedGrade(&encryptedGrade)

if err != nil {
log.WithError(err).Error("Could not save grade")
}
Expand Down Expand Up @@ -213,7 +211,6 @@ func sendGradesToDevice(device *model.IOSDevice, grades []model.IOSGrade, apns *
log.WithField("DeviceID", device.DeviceID).Info("Sending push notification")

_, err := apns.SendAlertNotification(notificationPayload)

if err != nil {
log.WithField("DeviceID", device.DeviceID).WithError(err).Error("Could not send notification")
}
Expand Down
1 change: 0 additions & 1 deletion server/backend/rpcserver.go
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,6 @@ func NewIOSNotificationsService() *IOSNotificationsService {
}

token, err := ios_apns_jwt.NewToken()

if err != nil {
log.WithError(err).Fatal("failed to create new token")
}
Expand Down
8 changes: 1 addition & 7 deletions server/model/ios_grade.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,13 +33,11 @@ type customDate struct {

func (c *customDate) UnmarshalXML(d *xml.Decoder, start xml.StartElement) error {
var v string
err := d.DecodeElement(&v, &start)
if err != nil {
if err := d.DecodeElement(&v, &start); err != nil {
return err
}

t, err := time.Parse("2006-01-02", v)

if err != nil {
return err
}
Expand All @@ -66,13 +64,11 @@ type IOSEncryptedGrade struct {

func (e *IOSEncryptedGrade) Encrypt(key string) error {
encryptedTitle, err := ios_crypto.SymmetricEncrypt(e.LectureTitle, key)

if err != nil {
return err
}

encryptedGrade, err := ios_crypto.SymmetricEncrypt(e.Grade, key)

if err != nil {
return err
}
Expand All @@ -86,13 +82,11 @@ func (e *IOSEncryptedGrade) Encrypt(key string) error {

func (e *IOSEncryptedGrade) Decrypt(key string) error {
decryptedTitle, err := ios_crypto.SymmetricDecrypt(ios_crypto.EncryptedString(e.LectureTitle), key)

if err != nil {
return err
}

decryptedGrade, err := ios_crypto.SymmetricDecrypt(ios_crypto.EncryptedString(e.Grade), key)

if err != nil {
return err
}
Expand Down

0 comments on commit 3533342

Please sign in to comment.