Compare commits
8 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
b5e098a694
|
|||
|
08fd34632a
|
|||
|
a7a2474e2a
|
|||
|
e15d70dd0e
|
|||
|
e98882a0c6
|
|||
|
24bf7cd434
|
|||
|
54c4f873fc
|
|||
|
b2de793758
|
@@ -205,7 +205,7 @@ class APIClient {
|
||||
fn: User.fromJson,
|
||||
authToken: auth.getToken(),
|
||||
query: {
|
||||
'confirm': ['true']
|
||||
'confirm': ['true'],
|
||||
},
|
||||
);
|
||||
}
|
||||
@@ -230,7 +230,7 @@ class APIClient {
|
||||
static Future<Client> updateClient(TokenSource auth, String clientID, {String? fcmToken, String? agentModel, String? name, String? agentVersion}) async {
|
||||
return await _request(
|
||||
name: 'updateClient',
|
||||
method: 'PUT',
|
||||
method: 'PATCH',
|
||||
relURL: 'users/${auth.getUserID()}/clients/$clientID',
|
||||
jsonBody: {
|
||||
if (fcmToken != null) 'fcm_token': fcmToken,
|
||||
@@ -259,7 +259,7 @@ class APIClient {
|
||||
method: 'GET',
|
||||
relURL: 'users/${auth.getUserID()}/channels',
|
||||
query: {
|
||||
'selector': [sel.apiKey]
|
||||
'selector': [sel.apiKey],
|
||||
},
|
||||
fn: (json) => ChannelWithSubscription.fromJsonArray(json['channels'] as List<dynamic>),
|
||||
authToken: auth.getToken(),
|
||||
|
||||
@@ -6,6 +6,7 @@ import 'package:flutter/material.dart';
|
||||
import 'package:flutter_local_notifications/flutter_local_notifications.dart';
|
||||
import 'package:in_app_purchase/in_app_purchase.dart';
|
||||
import 'package:provider/provider.dart';
|
||||
import 'package:simplecloudnotifier/api/api_client.dart';
|
||||
import 'package:simplecloudnotifier/main_messaging.dart';
|
||||
import 'package:simplecloudnotifier/main_utils.dart';
|
||||
import 'package:simplecloudnotifier/components/layout/nav_layout.dart';
|
||||
@@ -68,13 +69,15 @@ void main() async {
|
||||
print('[INIT] Request Notification permissions...');
|
||||
await FirebaseMessaging.instance.requestPermission(provisional: true);
|
||||
|
||||
FirebaseMessaging.instance.onTokenRefresh.listen((fcmToken) {
|
||||
FirebaseMessaging.instance.onTokenRefresh
|
||||
.listen((fcmToken) {
|
||||
try {
|
||||
setFirebaseToken(fcmToken);
|
||||
} catch (exc, trace) {
|
||||
ApplicationLog.error('Failed to set firebase token: ' + exc.toString(), trace: trace);
|
||||
}
|
||||
}).onError((dynamic err) {
|
||||
})
|
||||
.onError((dynamic err) {
|
||||
ApplicationLog.error('Failed to listen to token refresh events: ' + (err?.toString() ?? ''));
|
||||
});
|
||||
|
||||
@@ -96,6 +99,25 @@ void main() async {
|
||||
|
||||
await appAuth.tryMigrateFromV1();
|
||||
|
||||
if (appAuth.isAuth()) {
|
||||
print('[INIT] Load Client and potentially update...');
|
||||
|
||||
try {
|
||||
var client = await appAuth.loadClient(onlyCached: true);
|
||||
if (client != null) {
|
||||
if (client.agentModel != Globals().deviceModel || client.name != Globals().nameForClient() || client.agentVersion != Globals().version) {
|
||||
print('[INIT] Update Client info...');
|
||||
|
||||
final newClient = await APIClient.updateClient(appAuth, client.clientID, agentModel: Globals().deviceModel, name: Globals().nameForClient(), agentVersion: Globals().version);
|
||||
appAuth.setClientAndClientID(newClient);
|
||||
await appAuth.save();
|
||||
}
|
||||
}
|
||||
} catch (exc, trace) {
|
||||
ApplicationLog.error('Failed to get client (on init): ' + exc.toString(), trace: trace);
|
||||
}
|
||||
}
|
||||
|
||||
print('[INIT] Load Notifications...');
|
||||
|
||||
final flutterLocalNotificationsPlugin = FlutterLocalNotificationsPlugin();
|
||||
|
||||
@@ -40,11 +40,11 @@ void setFirebaseToken(String fcmToken) async {
|
||||
|
||||
if (client == null) {
|
||||
// should not really happen - perhaps someone externally deleted the client?
|
||||
final newClient = await APIClient.addClient(acc, fcmToken, Globals().deviceModel, Globals().version, Globals().hostname, Globals().clientType);
|
||||
final newClient = await APIClient.addClient(acc, fcmToken, Globals().deviceModel, Globals().version, Globals().nameForClient(), Globals().clientType);
|
||||
acc.setClientAndClientID(newClient);
|
||||
await acc.save();
|
||||
} else {
|
||||
final newClient = await APIClient.updateClient(acc, client.clientID, fcmToken: fcmToken, agentModel: Globals().deviceModel, name: Globals().hostname, agentVersion: Globals().version);
|
||||
final newClient = await APIClient.updateClient(acc, client.clientID, fcmToken: fcmToken, agentModel: Globals().deviceModel, name: Globals().nameForClient(), agentVersion: Globals().version);
|
||||
acc.setClientAndClientID(newClient);
|
||||
await acc.save();
|
||||
}
|
||||
|
||||
@@ -527,7 +527,7 @@ class _AccountRootPageState extends State<AccountRootPage> {
|
||||
|
||||
await Globals().setPrefFCMToken(fcmToken);
|
||||
|
||||
final user = await APIClient.createUserWithClient(null, fcmToken, Globals().platform, Globals().version, Globals().hostname, Globals().clientType);
|
||||
final user = await APIClient.createUserWithClient(null, fcmToken, Globals().platform, Globals().version, Globals().nameForClient(), Globals().clientType);
|
||||
|
||||
acc.set(user.user, user.clients[0], user.adminKey, user.sendKey);
|
||||
|
||||
|
||||
@@ -156,7 +156,7 @@ class _AccountLoginPageState extends State<AccountLoginPage> {
|
||||
|
||||
final user = await APIClient.getUser(DirectTokenSource(uid, atokv), uid);
|
||||
|
||||
final client = await APIClient.addClient(DirectTokenSource(uid, atokv), fcmToken, Globals().deviceModel, Globals().version, Globals().hostname, Globals().clientType);
|
||||
final client = await APIClient.addClient(DirectTokenSource(uid, atokv), fcmToken, Globals().deviceModel, Globals().version, Globals().nameForClient(), Globals().clientType);
|
||||
|
||||
acc.set(user, client, atokv, stokv);
|
||||
await acc.save();
|
||||
|
||||
@@ -12,6 +12,7 @@ import 'package:simplecloudnotifier/state/app_auth.dart';
|
||||
import 'package:simplecloudnotifier/state/application_log.dart';
|
||||
import 'package:simplecloudnotifier/utils/navi.dart';
|
||||
import 'package:simplecloudnotifier/utils/toaster.dart';
|
||||
import 'package:simplecloudnotifier/utils/dialogs.dart';
|
||||
import 'package:simplecloudnotifier/utils/ui.dart';
|
||||
|
||||
class ChannelScannerResultChannelSubscribe extends StatefulWidget {
|
||||
@@ -172,6 +173,38 @@ class _ChannelScannerResultChannelSubscribeState extends State<ChannelScannerRes
|
||||
|
||||
void _onSubscribe() async {
|
||||
final auth = Provider.of<AppAuth>(context, listen: false);
|
||||
|
||||
// Check if username is set
|
||||
try {
|
||||
final user = await auth.loadUser();
|
||||
if (user.username == null || user.username!.isEmpty) {
|
||||
// Show modal to set username
|
||||
var newusername = await UIDialogs.showUsernameRequiredDialog(context);
|
||||
|
||||
if (newusername == null) return; // User cancelled
|
||||
|
||||
newusername = newusername.trim();
|
||||
if (newusername.isEmpty) {
|
||||
Toaster.error("Error", 'Username cannot be empty');
|
||||
return;
|
||||
}
|
||||
|
||||
// Update username via API
|
||||
try {
|
||||
await APIClient.updateUser(auth, auth.userID!, username: newusername);
|
||||
await auth.loadUser(force: true); // Refresh cached user
|
||||
Toaster.success("Success", 'Username set');
|
||||
} catch (e) {
|
||||
Toaster.error("Error", 'Failed to set username: ${e.toString()}');
|
||||
return;
|
||||
}
|
||||
}
|
||||
} catch (e) {
|
||||
Toaster.error("Error", 'Failed to load user data: ${e.toString()}');
|
||||
return;
|
||||
}
|
||||
|
||||
// Proceed with subscription
|
||||
try {
|
||||
var sub = await APIClient.subscribeToChannelbyID(auth, widget.value.channelID, subscribeKey: widget.value.subscribeKey);
|
||||
if (sub.confirmed) {
|
||||
|
||||
@@ -5,6 +5,7 @@ import 'package:simplecloudnotifier/api/api_client.dart';
|
||||
import 'package:simplecloudnotifier/state/app_settings.dart';
|
||||
import 'package:simplecloudnotifier/state/app_auth.dart';
|
||||
import 'package:simplecloudnotifier/state/application_log.dart';
|
||||
import 'package:simplecloudnotifier/state/globals.dart';
|
||||
import 'package:simplecloudnotifier/utils/notifier.dart';
|
||||
import 'package:simplecloudnotifier/utils/toaster.dart';
|
||||
import 'package:simplecloudnotifier/utils/ui.dart';
|
||||
@@ -65,22 +66,31 @@ class _DebugActionsPageState extends State<DebugActionsPage> {
|
||||
onPressed: _sendTokenToServer,
|
||||
text: 'Send FCM Token to Server',
|
||||
),
|
||||
SizedBox(height: 4),
|
||||
UI.button(
|
||||
big: false,
|
||||
onPressed: _updateClient,
|
||||
text: 'Update Client on Server',
|
||||
),
|
||||
SizedBox(height: 20),
|
||||
UI.button(
|
||||
big: false,
|
||||
onPressed: () => Notifier.showLocalNotification('', 'TEST_CHANNEL', "Test Channel", "Channel for testing", "Hello World", "Local Notification test", null, null),
|
||||
text: 'Show local notification (generic)',
|
||||
),
|
||||
SizedBox(height: 4),
|
||||
UI.button(
|
||||
big: false,
|
||||
onPressed: () => Notifier.showLocalNotification('', 'TEST_CHANNEL', "Test Channel", "Channel for testing", "Hello World", "Local Notification test", null, 0),
|
||||
text: 'Show local notification (Prio = 0)',
|
||||
),
|
||||
SizedBox(height: 4),
|
||||
UI.button(
|
||||
big: false,
|
||||
onPressed: () => Notifier.showLocalNotification('', 'TEST_CHANNEL', "Test Channel", "Channel for testing", "Hello World", "Local Notification test", null, 1),
|
||||
text: 'Show local notification (Prio = 1)',
|
||||
),
|
||||
SizedBox(height: 4),
|
||||
UI.button(
|
||||
big: false,
|
||||
onPressed: () => Notifier.showLocalNotification('', 'TEST_CHANNEL', "Test Channel", "Channel for testing", "Hello World", "Local Notification test", null, 2),
|
||||
@@ -128,6 +138,26 @@ class _DebugActionsPageState extends State<DebugActionsPage> {
|
||||
}
|
||||
}
|
||||
|
||||
void _updateClient() async {
|
||||
try {
|
||||
final auth = AppAuth();
|
||||
|
||||
final clientID = auth.getClientID();
|
||||
if (clientID == null) {
|
||||
Toaster.error("Error", "No Client set");
|
||||
return;
|
||||
}
|
||||
|
||||
final newClient = await APIClient.updateClient(auth, clientID, agentModel: Globals().deviceModel, name: Globals().nameForClient(), agentVersion: Globals().version);
|
||||
auth.setClientAndClientID(newClient);
|
||||
|
||||
Toaster.success("Success", "Client updated");
|
||||
} catch (exc, trace) {
|
||||
Toaster.error("Error", "An error occurred while updating the client: ${exc.toString()}");
|
||||
ApplicationLog.error("An error occurred while updating the client: ${exc.toString()}", trace: trace);
|
||||
}
|
||||
}
|
||||
|
||||
void _copyToken() async {
|
||||
try {
|
||||
final fcmToken = await FirebaseMessaging.instance.getToken();
|
||||
|
||||
@@ -135,7 +135,7 @@ class _DebugRequestViewPageState extends State<DebugRequestViewPage> {
|
||||
void _copyCurl() {
|
||||
final method = '-X ${widget.request.method}';
|
||||
final header = widget.request.requestHeaders.entries.map((v) => '-H "${v.key}: ${v.value}"').join(' ');
|
||||
final body = widget.request.requestBody.isNotEmpty ? '-d "${widget.request.requestBody}"' : '';
|
||||
final body = widget.request.requestBody.isNotEmpty ? '-d \'${widget.request.requestBody}\'' : '';
|
||||
|
||||
final curlParts = ['curl', method, header, '"${widget.request.url}"', body];
|
||||
|
||||
|
||||
@@ -284,7 +284,9 @@ class _SendRootPageState extends State<SendRootPage> {
|
||||
}
|
||||
|
||||
try {
|
||||
await APIClient.sendMessage(acc.userID!, acc.tokenSend!, _msgContent.text);
|
||||
var content = (_msgContent.text != '') ? _msgContent.text : null;
|
||||
|
||||
await APIClient.sendMessage(acc.userID!, acc.tokenSend!, _msgTitle.text, content: content);
|
||||
Toaster.success("Success", 'Message sent');
|
||||
setState(() {
|
||||
_msgTitle.clear();
|
||||
@@ -306,7 +308,11 @@ class _SendRootPageState extends State<SendRootPage> {
|
||||
}
|
||||
|
||||
try {
|
||||
await APIClient.sendMessage(acc.userID!, acc.tokenSend!, _msgContent.text, channel: _channelName.text, senderName: _senderName.text, priority: _priority);
|
||||
var content = (_msgContent.text != '') ? _msgContent.text : null;
|
||||
var channel = (_channelName.text != '') ? _channelName.text : null;
|
||||
var sender = (_senderName.text != '') ? _senderName.text : null;
|
||||
|
||||
await APIClient.sendMessage(acc.userID!, acc.tokenSend!, _msgTitle.text, content: content, channel: channel, senderName: sender, priority: _priority);
|
||||
Toaster.success("Success", 'Message sent');
|
||||
setState(() {
|
||||
_msgTitle.clear();
|
||||
|
||||
@@ -99,7 +99,7 @@ class AppAuth extends ChangeNotifier implements TokenSource {
|
||||
|
||||
final user = await APIClient.getUser(DirectTokenSource(oldUserID, oldUserKey), oldUserID);
|
||||
|
||||
final client = await APIClient.addClient(DirectTokenSource(oldUserID, oldUserKey), fcmToken, Globals().deviceModel, Globals().version, Globals().hostname, Globals().clientType);
|
||||
final client = await APIClient.addClient(DirectTokenSource(oldUserID, oldUserKey), fcmToken, Globals().deviceModel, Globals().version, Globals().nameForClient(), Globals().clientType);
|
||||
|
||||
set(user, client, oldUserKey, newTokenSend.token);
|
||||
|
||||
@@ -232,7 +232,7 @@ class AppAuth extends ChangeNotifier implements TokenSource {
|
||||
return _user?.$1;
|
||||
}
|
||||
|
||||
Future<Client?> loadClient({bool force = false, Duration? forceIfOlder = null}) async {
|
||||
Future<Client?> loadClient({bool force = false, Duration? forceIfOlder = null, bool onlyCached = false}) async {
|
||||
if (forceIfOlder != null && _client != null && _client!.$2.difference(DateTime.now()) > forceIfOlder) {
|
||||
force = true;
|
||||
}
|
||||
@@ -245,6 +245,10 @@ class AppAuth extends ChangeNotifier implements TokenSource {
|
||||
throw Exception('Not authenticated');
|
||||
}
|
||||
|
||||
if (onlyCached) {
|
||||
return null;
|
||||
}
|
||||
|
||||
try {
|
||||
final client = await APIClient.getClient(this, _clientID!);
|
||||
|
||||
|
||||
@@ -92,4 +92,12 @@ class Globals {
|
||||
Future<bool> setPrefFCMToken(String value) {
|
||||
return sharedPrefs.setString("fcm.token", value);
|
||||
}
|
||||
|
||||
String nameForClient() {
|
||||
if (this.deviceName.isNotEmpty) {
|
||||
return this.deviceName;
|
||||
} else {
|
||||
return this.hostname;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -28,6 +28,40 @@ class UIDialogs {
|
||||
);
|
||||
}
|
||||
|
||||
static Future<String?> showUsernameRequiredDialog(BuildContext context) {
|
||||
var _textFieldController = TextEditingController();
|
||||
|
||||
return showDialog(
|
||||
context: context,
|
||||
builder: (context) => AlertDialog(
|
||||
title: Text('Username Required'),
|
||||
content: Column(
|
||||
mainAxisSize: MainAxisSize.min,
|
||||
crossAxisAlignment: CrossAxisAlignment.start,
|
||||
children: [
|
||||
Text('Please set a public username to subscribe to channels from other users.'),
|
||||
SizedBox(height: 16),
|
||||
TextField(
|
||||
autofocus: true,
|
||||
controller: _textFieldController,
|
||||
decoration: InputDecoration(hintText: 'Enter username'),
|
||||
),
|
||||
],
|
||||
),
|
||||
actions: [
|
||||
TextButton(
|
||||
onPressed: () => Navigator.of(context).pop(),
|
||||
child: Text('Cancel'),
|
||||
),
|
||||
TextButton(
|
||||
onPressed: () => Navigator.of(context).pop(_textFieldController.text),
|
||||
child: Text('OK'),
|
||||
),
|
||||
],
|
||||
),
|
||||
);
|
||||
}
|
||||
|
||||
static Future<bool> showConfirmDialog(BuildContext context, String title, {String? text, String? okText, String? cancelText}) {
|
||||
return showDialog<bool>(
|
||||
context: context,
|
||||
|
||||
@@ -2,7 +2,7 @@ name: simplecloudnotifier
|
||||
description: "Receive push messages"
|
||||
publish_to: 'none'
|
||||
|
||||
version: 2.1.1+509
|
||||
version: 2.2.2+544
|
||||
|
||||
environment:
|
||||
sdk: '>=3.9.0 <4.0.0'
|
||||
|
||||
2
scnserver/.gitignore
vendored
2
scnserver/.gitignore
vendored
@@ -17,8 +17,8 @@ simple_cloud_notifier-*.sql
|
||||
identifier.sqlite
|
||||
|
||||
.idea/dataSources.xml
|
||||
|
||||
.idea/copilot*
|
||||
.idea/go.imports.xml
|
||||
|
||||
.swaggobin
|
||||
|
||||
|
||||
@@ -1,14 +1,15 @@
|
||||
package handler
|
||||
|
||||
import (
|
||||
"database/sql"
|
||||
"errors"
|
||||
"net/http"
|
||||
|
||||
"blackforestbytes.com/simplecloudnotifier/api/apierr"
|
||||
"blackforestbytes.com/simplecloudnotifier/api/ginresp"
|
||||
"blackforestbytes.com/simplecloudnotifier/logic"
|
||||
"blackforestbytes.com/simplecloudnotifier/models"
|
||||
"database/sql"
|
||||
"errors"
|
||||
"git.blackforestbytes.com/BlackForestBytes/goext/ginext"
|
||||
"net/http"
|
||||
)
|
||||
|
||||
// GetUserPreview swaggerdoc
|
||||
@@ -52,7 +53,7 @@ func (h APIHandler) GetUserPreview(pctx ginext.PreContext) ginext.HTTPResponse {
|
||||
return ginresp.APIError(g, 500, apierr.DATABASE_ERROR, "Failed to query user", err)
|
||||
}
|
||||
|
||||
return finishSuccess(ginext.JSON(http.StatusOK, user.JSONPreview()))
|
||||
return finishSuccess(ginext.JSON(http.StatusOK, user.Preview()))
|
||||
|
||||
})
|
||||
}
|
||||
@@ -175,3 +176,65 @@ func (h APIHandler) GetUserKeyPreview(pctx ginext.PreContext) ginext.HTTPRespons
|
||||
|
||||
})
|
||||
}
|
||||
|
||||
// GetClientPreview swaggerdoc
|
||||
//
|
||||
// @Summary Get a client (similar to api-clients-get, but can be called from anyone and only returns a subset of fields)
|
||||
// @ID api-clients-get-preview
|
||||
// @Tags API-v2
|
||||
//
|
||||
// @Param cid path string true "ClientID"
|
||||
//
|
||||
// @Success 200 {object} handler.GetClientPreview.response
|
||||
//
|
||||
// @Failure 400 {object} ginresp.apiError "supplied values/parameters cannot be parsed / are invalid"
|
||||
// @Failure 401 {object} ginresp.apiError "user is not authorized / has missing permissions"
|
||||
// @Failure 404 {object} ginresp.apiError "client not found"
|
||||
// @Failure 500 {object} ginresp.apiError "internal server error"
|
||||
//
|
||||
// @Router /api/v2/preview/clients/{cid} [GET]
|
||||
func (h APIHandler) GetClientPreview(pctx ginext.PreContext) ginext.HTTPResponse {
|
||||
type uri struct {
|
||||
ClientID models.ClientID `uri:"cid" binding:"entityid"`
|
||||
}
|
||||
type response struct {
|
||||
Client models.ClientPreview `json:"client"`
|
||||
User models.UserPreview `json:"user"`
|
||||
}
|
||||
|
||||
var u uri
|
||||
ctx, g, errResp := pctx.URI(&u).Start()
|
||||
if errResp != nil {
|
||||
return *errResp
|
||||
}
|
||||
defer ctx.Cancel()
|
||||
|
||||
return h.app.DoRequest(ctx, g, models.TLockRead, func(ctx *logic.AppContext, finishSuccess func(r ginext.HTTPResponse) ginext.HTTPResponse) ginext.HTTPResponse {
|
||||
|
||||
if permResp := ctx.CheckPermissionAny(); permResp != nil {
|
||||
return *permResp
|
||||
}
|
||||
|
||||
client, err := h.database.GetClientByID(ctx, u.ClientID)
|
||||
if client == nil {
|
||||
return ginresp.APIError(g, 404, apierr.CLIENT_NOT_FOUND, "Client not found", err)
|
||||
}
|
||||
if err != nil {
|
||||
return ginresp.APIError(g, 500, apierr.DATABASE_ERROR, "Failed to query client", err)
|
||||
}
|
||||
|
||||
user, err := h.database.GetUser(ctx, client.UserID)
|
||||
if errors.Is(err, sql.ErrNoRows) {
|
||||
return ginresp.APIError(g, 404, apierr.USER_NOT_FOUND, "User not found", err)
|
||||
}
|
||||
if err != nil {
|
||||
return ginresp.APIError(g, 500, apierr.DATABASE_ERROR, "Failed to query user", err)
|
||||
}
|
||||
|
||||
return finishSuccess(ginext.JSON(http.StatusOK, response{
|
||||
Client: client.Preview(),
|
||||
User: user.Preview(),
|
||||
}))
|
||||
|
||||
})
|
||||
}
|
||||
|
||||
@@ -171,6 +171,7 @@ func (r *Router) Init(e *ginext.GinWrapper) error {
|
||||
apiv2.GET("/preview/users/:uid").Handle(r.apiHandler.GetUserPreview)
|
||||
apiv2.GET("/preview/keys/:kid").Handle(r.apiHandler.GetUserKeyPreview)
|
||||
apiv2.GET("/preview/channels/:cid").Handle(r.apiHandler.GetChannelPreview)
|
||||
apiv2.GET("/preview/clients/:cid").Handle(r.apiHandler.GetClientPreview)
|
||||
}
|
||||
|
||||
// ================ Send API (unversioned) ================
|
||||
|
||||
@@ -53,6 +53,15 @@ func (db *Database) GetClient(ctx db.TxContext, userid models.UserID, clientid m
|
||||
}, sq.SModeExtended, sq.Safe)
|
||||
}
|
||||
|
||||
func (db *Database) GetClientByID(ctx db.TxContext, clientid models.ClientID) (*models.Client, error) {
|
||||
tx, err := ctx.GetOrCreateTransaction(db)
|
||||
if err != nil {
|
||||
return nil, err
|
||||
}
|
||||
|
||||
return sq.QuerySingleOpt[models.Client](ctx, tx, "SELECT * FROM clients WHERE deleted=0 AND client_id = :cid LIMIT 1", sq.PP{"cid": clientid}, sq.SModeExtended, sq.Safe)
|
||||
}
|
||||
|
||||
func (db *Database) GetClientOpt(ctx db.TxContext, userid models.UserID, clientid models.ClientID) (*models.Client, error) {
|
||||
tx, err := ctx.GetOrCreateTransaction(db)
|
||||
if err != nil {
|
||||
|
||||
@@ -21,3 +21,25 @@ type Client struct {
|
||||
Name *string `db:"name" json:"name"`
|
||||
Deleted bool `db:"deleted" json:"-"`
|
||||
}
|
||||
|
||||
type ClientPreview struct {
|
||||
ClientID ClientID `json:"client_id"`
|
||||
UserID UserID `json:"user_id"`
|
||||
Type ClientType `json:"type"`
|
||||
TimestampCreated SCNTime `json:"timestamp_created"`
|
||||
AgentModel string `json:"agent_model"`
|
||||
AgentVersion string `json:"agent_version"`
|
||||
Name *string `json:"name"`
|
||||
}
|
||||
|
||||
func (c Client) Preview() ClientPreview {
|
||||
return ClientPreview{
|
||||
ClientID: c.ClientID,
|
||||
UserID: c.UserID,
|
||||
Type: c.Type,
|
||||
TimestampCreated: c.TimestampCreated,
|
||||
AgentModel: c.AgentModel,
|
||||
AgentVersion: c.AgentVersion,
|
||||
Name: c.Name,
|
||||
}
|
||||
}
|
||||
|
||||
@@ -88,9 +88,9 @@ func (u User) MaxTitleLength() int {
|
||||
|
||||
func (u User) QuotaPerDay() int {
|
||||
if u.IsPro {
|
||||
return 5000
|
||||
return 15_000
|
||||
} else {
|
||||
return 50
|
||||
return 500
|
||||
}
|
||||
}
|
||||
|
||||
@@ -135,7 +135,7 @@ func (u User) MaxTimestampDiffHours() int {
|
||||
return 24
|
||||
}
|
||||
|
||||
func (u User) JSONPreview() UserPreview {
|
||||
func (u User) Preview() UserPreview {
|
||||
return UserPreview{
|
||||
UserID: u.UserID,
|
||||
Username: u.Username,
|
||||
|
||||
@@ -1,22 +1,23 @@
|
||||
package push
|
||||
|
||||
import (
|
||||
scn "blackforestbytes.com/simplecloudnotifier"
|
||||
"blackforestbytes.com/simplecloudnotifier/models"
|
||||
"bytes"
|
||||
"context"
|
||||
_ "embed"
|
||||
"encoding/json"
|
||||
"errors"
|
||||
"fmt"
|
||||
"git.blackforestbytes.com/BlackForestBytes/goext/langext"
|
||||
"github.com/gin-gonic/gin"
|
||||
"github.com/rs/zerolog/log"
|
||||
"io"
|
||||
"net/http"
|
||||
"strconv"
|
||||
"strings"
|
||||
"time"
|
||||
|
||||
scn "blackforestbytes.com/simplecloudnotifier"
|
||||
"blackforestbytes.com/simplecloudnotifier/models"
|
||||
"git.blackforestbytes.com/BlackForestBytes/goext/langext"
|
||||
"github.com/gin-gonic/gin"
|
||||
"github.com/rs/zerolog/log"
|
||||
)
|
||||
|
||||
// https://firebase.google.com/docs/cloud-messaging/send-message#rest
|
||||
@@ -66,7 +67,13 @@ func (fb FirebaseConnector) SendNotification(ctx context.Context, user models.Us
|
||||
"title": msg.Title,
|
||||
"body": msg.ShortContent(),
|
||||
},
|
||||
"apns": gin.H{},
|
||||
"apns": gin.H{
|
||||
"payload": gin.H{
|
||||
"aps": gin.H{
|
||||
"sound": "default",
|
||||
},
|
||||
},
|
||||
},
|
||||
}
|
||||
} else if client.Type == models.ClientTypeAndroid {
|
||||
jsonBody = gin.H{
|
||||
|
||||
@@ -1,13 +1,14 @@
|
||||
package test
|
||||
|
||||
import (
|
||||
"blackforestbytes.com/simplecloudnotifier/push"
|
||||
tt "blackforestbytes.com/simplecloudnotifier/test/util"
|
||||
"fmt"
|
||||
"github.com/gin-gonic/gin"
|
||||
"net/url"
|
||||
"testing"
|
||||
"time"
|
||||
|
||||
"blackforestbytes.com/simplecloudnotifier/push"
|
||||
tt "blackforestbytes.com/simplecloudnotifier/test/util"
|
||||
"github.com/gin-gonic/gin"
|
||||
)
|
||||
|
||||
func TestSendCompatWithOldUser(t *testing.T) {
|
||||
@@ -309,7 +310,7 @@ func TestCompatRegister(t *testing.T) {
|
||||
tt.AssertEqual(t, "success", true, r0["success"])
|
||||
tt.AssertEqual(t, "message", "New user registered", r0["message"])
|
||||
tt.AssertEqual(t, "quota", 0, r0["quota"])
|
||||
tt.AssertEqual(t, "quota_max", 50, r0["quota_max"])
|
||||
tt.AssertEqual(t, "quota_max", 500, r0["quota_max"])
|
||||
tt.AssertEqual(t, "is_pro", false, r0["is_pro"])
|
||||
}
|
||||
|
||||
@@ -321,7 +322,7 @@ func TestCompatRegisterPro(t *testing.T) {
|
||||
tt.AssertEqual(t, "success", true, r0["success"])
|
||||
tt.AssertEqual(t, "message", "New user registered", r0["message"])
|
||||
tt.AssertEqual(t, "quota", 0, r0["quota"])
|
||||
tt.AssertEqual(t, "quota_max", 5000, r0["quota_max"])
|
||||
tt.AssertEqual(t, "quota_max", 15000, r0["quota_max"])
|
||||
tt.AssertEqual(t, "is_pro", true, r0["is_pro"])
|
||||
|
||||
r1 := tt.RequestGet[gin.H](t, baseUrl, fmt.Sprintf("/api/register.php?fcm_token=%s&pro=%s&pro_token=%s", "DUMMY_FCM", "true", url.QueryEscape("INVALID")))
|
||||
@@ -345,7 +346,7 @@ func TestCompatInfo(t *testing.T) {
|
||||
tt.AssertEqual(t, "is_pro", 0, r1["is_pro"])
|
||||
tt.AssertEqual(t, "message", "ok", r1["message"])
|
||||
tt.AssertEqual(t, "quota", 0, r1["quota"])
|
||||
tt.AssertEqual(t, "quota_max", 50, r1["quota_max"])
|
||||
tt.AssertEqual(t, "quota_max", 500, r1["quota_max"])
|
||||
tt.AssertEqual(t, "unack_count", 0, r1["unack_count"])
|
||||
tt.AssertEqual(t, "user_id", userid, r1["user_id"])
|
||||
tt.AssertEqual(t, "user_key", userkey, r1["user_key"])
|
||||
@@ -363,7 +364,7 @@ func TestCompatInfo(t *testing.T) {
|
||||
tt.AssertEqual(t, "is_pro", 0, r2["is_pro"])
|
||||
tt.AssertEqual(t, "message", "ok", r2["message"])
|
||||
tt.AssertEqual(t, "quota", 1, r2["quota"])
|
||||
tt.AssertEqual(t, "quota_max", 50, r2["quota_max"])
|
||||
tt.AssertEqual(t, "quota_max", 500, r2["quota_max"])
|
||||
tt.AssertEqual(t, "unack_count", 1, r2["unack_count"])
|
||||
tt.AssertEqual(t, "user_id", userid, r2["user_id"])
|
||||
tt.AssertEqual(t, "user_key", userkey, r2["user_key"])
|
||||
@@ -490,7 +491,7 @@ func TestCompatUpdateUserKey(t *testing.T) {
|
||||
tt.AssertEqual(t, "is_pro", 0, r1["is_pro"])
|
||||
tt.AssertEqual(t, "message", "ok", r1["message"])
|
||||
tt.AssertEqual(t, "quota", 1, r1["quota"])
|
||||
tt.AssertEqual(t, "quota_max", 50, r1["quota_max"])
|
||||
tt.AssertEqual(t, "quota_max", 500, r1["quota_max"])
|
||||
tt.AssertEqual(t, "unack_count", 1, r1["unack_count"])
|
||||
tt.AssertEqual(t, "user_id", userid, r1["user_id"])
|
||||
tt.AssertEqual(t, "user_key", newkey, r1["user_key"])
|
||||
@@ -527,7 +528,7 @@ func TestCompatUpdateFCM(t *testing.T) {
|
||||
tt.AssertEqual(t, "is_pro", 0, r1["is_pro"])
|
||||
tt.AssertEqual(t, "message", "ok", r1["message"])
|
||||
tt.AssertEqual(t, "quota", 1, r1["quota"])
|
||||
tt.AssertEqual(t, "quota_max", 50, r1["quota_max"])
|
||||
tt.AssertEqual(t, "quota_max", 500, r1["quota_max"])
|
||||
tt.AssertEqual(t, "unack_count", 1, r1["unack_count"])
|
||||
tt.AssertEqual(t, "user_id", userid, r1["user_id"])
|
||||
tt.AssertEqual(t, "user_key", newkey, r1["user_key"])
|
||||
@@ -554,7 +555,7 @@ func TestCompatUpgrade(t *testing.T) {
|
||||
tt.AssertEqual(t, "success", true, r0["success"])
|
||||
tt.AssertEqual(t, "message", "New user registered", r0["message"])
|
||||
tt.AssertEqual(t, "quota", 0, r0["quota"])
|
||||
tt.AssertEqual(t, "quota_max", 50, r0["quota_max"])
|
||||
tt.AssertEqual(t, "quota_max", 500, r0["quota_max"])
|
||||
tt.AssertEqual(t, "is_pro", false, r0["is_pro"])
|
||||
|
||||
userid := int64(r0["user_id"].(float64))
|
||||
@@ -564,7 +565,7 @@ func TestCompatUpgrade(t *testing.T) {
|
||||
tt.AssertEqual(t, "success", true, r1["success"])
|
||||
tt.AssertEqual(t, "message", "user updated", r1["message"])
|
||||
tt.AssertEqual(t, "quota", 0, r1["quota"])
|
||||
tt.AssertEqual(t, "quota_max", 5000, r1["quota_max"])
|
||||
tt.AssertEqual(t, "quota_max", 15000, r1["quota_max"])
|
||||
tt.AssertEqual(t, "is_pro", true, r1["is_pro"])
|
||||
}
|
||||
|
||||
|
||||
@@ -1,14 +1,15 @@
|
||||
package test
|
||||
|
||||
import (
|
||||
ct "blackforestbytes.com/simplecloudnotifier/db/cursortoken"
|
||||
"blackforestbytes.com/simplecloudnotifier/models"
|
||||
tt "blackforestbytes.com/simplecloudnotifier/test/util"
|
||||
"fmt"
|
||||
"github.com/gin-gonic/gin"
|
||||
"net/url"
|
||||
"testing"
|
||||
"time"
|
||||
|
||||
ct "blackforestbytes.com/simplecloudnotifier/db/cursortoken"
|
||||
"blackforestbytes.com/simplecloudnotifier/models"
|
||||
tt "blackforestbytes.com/simplecloudnotifier/test/util"
|
||||
"github.com/gin-gonic/gin"
|
||||
)
|
||||
|
||||
func TestRequestLogSimple(t *testing.T) {
|
||||
@@ -126,6 +127,8 @@ func TestRequestLogSimple(t *testing.T) {
|
||||
}
|
||||
|
||||
func TestRequestLogAPI(t *testing.T) {
|
||||
t.Skip("Flaky test - and kinda hacky")
|
||||
|
||||
ws, baseUrl, stop := tt.StartSimpleWebserver(t)
|
||||
defer stop()
|
||||
|
||||
|
||||
@@ -1174,8 +1174,8 @@ func TestQuotaExceededNoPro(t *testing.T) {
|
||||
sendtok := r0["send_key"].(string)
|
||||
|
||||
tt.AssertStrRepEqual(t, "quota.0", 0, r0["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.0", 50, r0["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.0", 50, r0["quota_remaining"])
|
||||
tt.AssertStrRepEqual(t, "quota.0", 500, r0["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.0", 500, r0["quota_remaining"])
|
||||
|
||||
{
|
||||
msg1 := tt.RequestPost[gin.H](t, baseUrl, "/", gin.H{
|
||||
@@ -1184,18 +1184,18 @@ func TestQuotaExceededNoPro(t *testing.T) {
|
||||
"title": tt.ShortLipsum0(2),
|
||||
})
|
||||
tt.AssertStrRepEqual(t, "quota.msg.1", 1, msg1["quota"])
|
||||
tt.AssertStrRepEqual(t, "quota.msg.1", 50, msg1["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.msg.1", 500, msg1["quota_max"])
|
||||
}
|
||||
|
||||
{
|
||||
usr := tt.RequestAuthGet[gin.H](t, admintok, baseUrl, fmt.Sprintf("/api/v2/users/%s", uid))
|
||||
|
||||
tt.AssertStrRepEqual(t, "quota.1", 1, usr["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.1", 50, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.1", 49, usr["quota_remaining"])
|
||||
tt.AssertStrRepEqual(t, "quota.1", 500, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.1", 499, usr["quota_remaining"])
|
||||
}
|
||||
|
||||
for i := 0; i < 48; i++ {
|
||||
for i := 0; i < 498; i++ {
|
||||
|
||||
tt.RequestPost[gin.H](t, baseUrl, "/", gin.H{
|
||||
"key": sendtok,
|
||||
@@ -1207,24 +1207,24 @@ func TestQuotaExceededNoPro(t *testing.T) {
|
||||
{
|
||||
usr := tt.RequestAuthGet[gin.H](t, admintok, baseUrl, fmt.Sprintf("/api/v2/users/%s", uid))
|
||||
|
||||
tt.AssertStrRepEqual(t, "quota.49", 49, usr["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.49", 50, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.49", 499, usr["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.49", 500, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.49", 1, usr["quota_remaining"])
|
||||
}
|
||||
|
||||
msg50 := tt.RequestPost[gin.H](t, baseUrl, "/", gin.H{
|
||||
msg500 := tt.RequestPost[gin.H](t, baseUrl, "/", gin.H{
|
||||
"key": sendtok,
|
||||
"user_id": uid,
|
||||
"title": tt.ShortLipsum0(2),
|
||||
})
|
||||
tt.AssertStrRepEqual(t, "quota.msg.50", 50, msg50["quota"])
|
||||
tt.AssertStrRepEqual(t, "quota.msg.50", 50, msg50["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.msg.50", 500, msg500["quota"])
|
||||
tt.AssertStrRepEqual(t, "quota.msg.50", 500, msg500["quota_max"])
|
||||
|
||||
{
|
||||
usr := tt.RequestAuthGet[gin.H](t, admintok, baseUrl, fmt.Sprintf("/api/v2/users/%s", uid))
|
||||
|
||||
tt.AssertStrRepEqual(t, "quota.50", 50, usr["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.50", 50, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.50", 500, usr["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.50", 500, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.50", 0, usr["quota_remaining"])
|
||||
}
|
||||
|
||||
@@ -1252,8 +1252,8 @@ func TestQuotaExceededPro(t *testing.T) {
|
||||
sendtok := r0["send_key"].(string)
|
||||
|
||||
tt.AssertStrRepEqual(t, "quota.0", 0, r0["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.0", 5000, r0["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.0", 5000, r0["quota_remaining"])
|
||||
tt.AssertStrRepEqual(t, "quota.0", 15000, r0["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.0", 15000, r0["quota_remaining"])
|
||||
|
||||
{
|
||||
msg1 := tt.RequestPost[gin.H](t, baseUrl, "/", gin.H{
|
||||
@@ -1262,18 +1262,18 @@ func TestQuotaExceededPro(t *testing.T) {
|
||||
"title": tt.ShortLipsum0(2),
|
||||
})
|
||||
tt.AssertStrRepEqual(t, "quota.msg.1", 1, msg1["quota"])
|
||||
tt.AssertStrRepEqual(t, "quota.msg.1", 5000, msg1["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.msg.1", 15000, msg1["quota_max"])
|
||||
}
|
||||
|
||||
{
|
||||
usr := tt.RequestAuthGet[gin.H](t, admintok, baseUrl, fmt.Sprintf("/api/v2/users/%s", uid))
|
||||
|
||||
tt.AssertStrRepEqual(t, "quota.1", 1, usr["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.1", 5000, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.1", 4999, usr["quota_remaining"])
|
||||
tt.AssertStrRepEqual(t, "quota.1", 15000, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.1", 14999, usr["quota_remaining"])
|
||||
}
|
||||
|
||||
for i := 0; i < 4998; i++ {
|
||||
for i := 0; i < 14998; i++ {
|
||||
|
||||
tt.RequestPost[gin.H](t, baseUrl, "/", gin.H{
|
||||
"key": sendtok,
|
||||
@@ -1285,9 +1285,9 @@ func TestQuotaExceededPro(t *testing.T) {
|
||||
{
|
||||
usr := tt.RequestAuthGet[gin.H](t, admintok, baseUrl, fmt.Sprintf("/api/v2/users/%s", uid))
|
||||
|
||||
tt.AssertStrRepEqual(t, "quota.999", 4999, usr["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.999", 5000, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.999", 1, usr["quota_remaining"])
|
||||
tt.AssertStrRepEqual(t, "quota.14999", 14999, usr["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.14999", 15000, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.14999", 1, usr["quota_remaining"])
|
||||
}
|
||||
|
||||
msg50 := tt.RequestPost[gin.H](t, baseUrl, "/", gin.H{
|
||||
@@ -1295,14 +1295,14 @@ func TestQuotaExceededPro(t *testing.T) {
|
||||
"user_id": uid,
|
||||
"title": tt.ShortLipsum0(2),
|
||||
})
|
||||
tt.AssertStrRepEqual(t, "quota.msg.5000", 5000, msg50["quota"])
|
||||
tt.AssertStrRepEqual(t, "quota.msg.5000", 5000, msg50["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.msg.5000", 15000, msg50["quota"])
|
||||
tt.AssertStrRepEqual(t, "quota.msg.5000", 15000, msg50["quota_max"])
|
||||
|
||||
{
|
||||
usr := tt.RequestAuthGet[gin.H](t, admintok, baseUrl, fmt.Sprintf("/api/v2/users/%s", uid))
|
||||
|
||||
tt.AssertStrRepEqual(t, "quota.5000", 5000, usr["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.5000", 5000, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.5000", 15000, usr["quota_used"])
|
||||
tt.AssertStrRepEqual(t, "quota.5000", 15000, usr["quota_max"])
|
||||
tt.AssertStrRepEqual(t, "quota.5000", 0, usr["quota_remaining"])
|
||||
}
|
||||
|
||||
|
||||
@@ -1,11 +1,12 @@
|
||||
package test
|
||||
|
||||
import (
|
||||
"fmt"
|
||||
"testing"
|
||||
|
||||
"blackforestbytes.com/simplecloudnotifier/api/apierr"
|
||||
tt "blackforestbytes.com/simplecloudnotifier/test/util"
|
||||
"fmt"
|
||||
"github.com/gin-gonic/gin"
|
||||
"testing"
|
||||
)
|
||||
|
||||
func TestCreateUserNoClient(t *testing.T) {
|
||||
@@ -451,8 +452,8 @@ func TestGetUserNoPro(t *testing.T) {
|
||||
tt.AssertEqual(t, "timestamp_lastsent", nil, r1["timestamp_lastsent"])
|
||||
tt.AssertEqual(t, "messages_sent", "0", fmt.Sprintf("%v", r1["messages_sent"]))
|
||||
tt.AssertEqual(t, "quota_used", "0", fmt.Sprintf("%v", r1["quota_used"]))
|
||||
tt.AssertEqual(t, "quota_remaining", "50", fmt.Sprintf("%v", r1["quota_remaining"]))
|
||||
tt.AssertEqual(t, "quota_max", "50", fmt.Sprintf("%v", r1["quota_max"]))
|
||||
tt.AssertEqual(t, "quota_remaining", "500", fmt.Sprintf("%v", r1["quota_remaining"]))
|
||||
tt.AssertEqual(t, "quota_max", "500", fmt.Sprintf("%v", r1["quota_max"]))
|
||||
tt.AssertEqual(t, "is_pro", "false", fmt.Sprintf("%v", r1["is_pro"]))
|
||||
tt.AssertEqual(t, "default_channel", "main", fmt.Sprintf("%v", r1["default_channel"]))
|
||||
tt.AssertEqual(t, "max_body_size", "2048", fmt.Sprintf("%v", r1["max_body_size"]))
|
||||
@@ -485,8 +486,8 @@ func TestGetUserPro(t *testing.T) {
|
||||
tt.AssertEqual(t, "timestamp_lastsent", nil, r1["timestamp_lastsent"])
|
||||
tt.AssertEqual(t, "messages_sent", "0", fmt.Sprintf("%v", r1["messages_sent"]))
|
||||
tt.AssertEqual(t, "quota_used", "0", fmt.Sprintf("%v", r1["quota_used"]))
|
||||
tt.AssertEqual(t, "quota_remaining", "5000", fmt.Sprintf("%v", r1["quota_remaining"]))
|
||||
tt.AssertEqual(t, "quota_max", "5000", fmt.Sprintf("%v", r1["quota_max"]))
|
||||
tt.AssertEqual(t, "quota_remaining", "15000", fmt.Sprintf("%v", r1["quota_remaining"]))
|
||||
tt.AssertEqual(t, "quota_max", "15000", fmt.Sprintf("%v", r1["quota_max"]))
|
||||
tt.AssertEqual(t, "is_pro", "true", fmt.Sprintf("%v", r1["is_pro"]))
|
||||
tt.AssertEqual(t, "default_channel", "main", fmt.Sprintf("%v", r1["default_channel"]))
|
||||
tt.AssertEqual(t, "max_body_size", "2097152", fmt.Sprintf("%d", (int64)(r1["max_body_size"].(float64))))
|
||||
|
||||
@@ -3,13 +3,14 @@ package util
|
||||
import (
|
||||
"encoding/json"
|
||||
"fmt"
|
||||
"git.blackforestbytes.com/BlackForestBytes/goext/langext"
|
||||
"github.com/gin-gonic/gin"
|
||||
"math"
|
||||
"reflect"
|
||||
"runtime/debug"
|
||||
"strings"
|
||||
"testing"
|
||||
|
||||
"git.blackforestbytes.com/BlackForestBytes/goext/langext"
|
||||
"github.com/gin-gonic/gin"
|
||||
)
|
||||
|
||||
func AssertJsonMapEqual(t *testing.T, key string, expected map[string]any, actual map[string]any) {
|
||||
@@ -36,6 +37,7 @@ func AssertJsonMapEqual(t *testing.T, key string, expected map[string]any, actua
|
||||
}
|
||||
|
||||
func AssertEqual(t *testing.T, key string, expected any, actual any) {
|
||||
t.Helper()
|
||||
|
||||
// try to fix types, kinda hacky, but its only unit tests...
|
||||
switch vex := expected.(type) {
|
||||
|
||||
@@ -1,3 +1,5 @@
|
||||
import { UserPreview } from "./user.model";
|
||||
|
||||
export type ClientType = 'ANDROID' | 'IOS' | 'LINUX' | 'MACOS' | 'WINDOWS';
|
||||
|
||||
export interface Client {
|
||||
@@ -15,6 +17,19 @@ export interface ClientListResponse {
|
||||
clients: Client[];
|
||||
}
|
||||
|
||||
export interface ClientPreview {
|
||||
client_id: string;
|
||||
name: string | null;
|
||||
type: ClientType;
|
||||
agent_model: string;
|
||||
agent_version: string;
|
||||
}
|
||||
|
||||
export interface ClientPreviewResponse {
|
||||
user: UserPreview;
|
||||
client: ClientPreview;
|
||||
}
|
||||
|
||||
export function getClientTypeIcon(type: ClientType): string {
|
||||
switch (type) {
|
||||
case 'ANDROID':
|
||||
|
||||
@@ -26,6 +26,7 @@ import {
|
||||
UpdateKeyRequest,
|
||||
Client,
|
||||
ClientListResponse,
|
||||
ClientPreviewResponse,
|
||||
SenderNameStatistics,
|
||||
SenderNameListResponse,
|
||||
DeliveryListResponse,
|
||||
@@ -93,6 +94,10 @@ export class ApiService {
|
||||
return this.http.delete<Client>(`${this.baseUrl}/users/${userId}/clients/${clientId}`);
|
||||
}
|
||||
|
||||
getClientPreview(clientId: string): Observable<ClientPreviewResponse> {
|
||||
return this.http.get<ClientPreviewResponse>(`${this.baseUrl}/preview/clients/${clientId}`);
|
||||
}
|
||||
|
||||
// Channel endpoints
|
||||
getChannels(userId: string, selector?: ChannelSelector): Observable<ChannelListResponse> {
|
||||
let params = new HttpParams();
|
||||
|
||||
44
webapp/src/app/core/services/client-cache.service.ts
Normal file
44
webapp/src/app/core/services/client-cache.service.ts
Normal file
@@ -0,0 +1,44 @@
|
||||
import { Injectable, inject } from '@angular/core';
|
||||
import { Observable, of, catchError, map, shareReplay } from 'rxjs';
|
||||
import { ApiService } from './api.service';
|
||||
|
||||
export interface ResolvedClient {
|
||||
clientId: string;
|
||||
clientName: string | null;
|
||||
userId: string;
|
||||
userName: string | null;
|
||||
agentModel: string;
|
||||
agentVersion: string;
|
||||
}
|
||||
|
||||
@Injectable({
|
||||
providedIn: 'root'
|
||||
})
|
||||
export class ClientCacheService {
|
||||
private apiService = inject(ApiService);
|
||||
|
||||
private cache = new Map<string, Observable<ResolvedClient | null>>();
|
||||
|
||||
resolveClient(clientId: string): Observable<ResolvedClient | null> {
|
||||
if (!this.cache.has(clientId)) {
|
||||
const request$ = this.apiService.getClientPreview(clientId).pipe(
|
||||
map(response => ({
|
||||
clientId: response.client.client_id,
|
||||
clientName: response.client.name,
|
||||
userId: response.user.user_id,
|
||||
userName: response.user.username,
|
||||
agentModel: response.client.agent_model,
|
||||
agentVersion: response.client.agent_version,
|
||||
})),
|
||||
catchError(() => of(null)),
|
||||
shareReplay(1)
|
||||
);
|
||||
this.cache.set(clientId, request$);
|
||||
}
|
||||
return this.cache.get(clientId)!;
|
||||
}
|
||||
|
||||
clearCache(): void {
|
||||
this.cache.clear();
|
||||
}
|
||||
}
|
||||
@@ -90,18 +90,33 @@
|
||||
>
|
||||
<thead>
|
||||
<tr>
|
||||
<th>Client ID</th>
|
||||
<th>Client</th>
|
||||
<th>User</th>
|
||||
<th>Agent</th>
|
||||
<th>Status</th>
|
||||
<th>Retries</th>
|
||||
<th>Created</th>
|
||||
<th>Finalized</th>
|
||||
<th>FCM-ID</th>
|
||||
</tr>
|
||||
</thead>
|
||||
<tbody>
|
||||
@for (delivery of deliveriesTable.data; track delivery.delivery_id) {
|
||||
<tr>
|
||||
<td>
|
||||
<span class="mono">{{ delivery.receiver_client_id }}</span>
|
||||
<div class="cell-name">{{ getResolvedClient(delivery.receiver_client_id)?.clientName || '-' }}</div>
|
||||
<div class="cell-id mono">{{ delivery.receiver_client_id }}</div>
|
||||
</td>
|
||||
<td>
|
||||
<div class="cell-name">{{ getResolvedClient(delivery.receiver_client_id)?.userName || '-' }}</div>
|
||||
<div class="cell-id mono">{{ delivery.receiver_user_id }}</div>
|
||||
</td>
|
||||
<td>
|
||||
@if (getResolvedClient(delivery.receiver_client_id); as client) {
|
||||
{{ client.agentModel }} {{ client.agentVersion }}
|
||||
} @else {
|
||||
-
|
||||
}
|
||||
</td>
|
||||
<td>
|
||||
<nz-tag [nzColor]="getStatusColor(delivery.status)">
|
||||
@@ -111,6 +126,20 @@
|
||||
<td>{{ delivery.retry_count }}</td>
|
||||
<td>{{ delivery.timestamp_created | date:'yyyy-MM-dd HH:mm:ss' }}</td>
|
||||
<td>{{ delivery.timestamp_finalized ? (delivery.timestamp_finalized | date:'yyyy-MM-dd HH:mm:ss') : '-' }}</td>
|
||||
<td>
|
||||
@if (delivery.fcm_message_id) {
|
||||
<span
|
||||
nz-icon
|
||||
nzType="copy"
|
||||
class="action-icon"
|
||||
nz-tooltip
|
||||
nzTooltipTitle="Copy FCM ID"
|
||||
[appCopyToClipboard]="delivery.fcm_message_id"
|
||||
></span>
|
||||
} @else {
|
||||
-
|
||||
}
|
||||
</td>
|
||||
</tr>
|
||||
}
|
||||
</tbody>
|
||||
|
||||
@@ -8,13 +8,16 @@ import { NzTagModule } from 'ng-zorro-antd/tag';
|
||||
import { NzSpinModule } from 'ng-zorro-antd/spin';
|
||||
import { NzPopconfirmModule } from 'ng-zorro-antd/popconfirm';
|
||||
import { NzTableModule } from 'ng-zorro-antd/table';
|
||||
import { NzToolTipModule } from 'ng-zorro-antd/tooltip';
|
||||
import { ApiService } from '../../../core/services/api.service';
|
||||
import { AuthService } from '../../../core/services/auth.service';
|
||||
import { NotificationService } from '../../../core/services/notification.service';
|
||||
import { SettingsService } from '../../../core/services/settings.service';
|
||||
import { KeyCacheService, ResolvedKey } from '../../../core/services/key-cache.service';
|
||||
import { UserCacheService, ResolvedUser } from '../../../core/services/user-cache.service';
|
||||
import { ClientCacheService, ResolvedClient } from '../../../core/services/client-cache.service';
|
||||
import { Message, Delivery } from '../../../core/models';
|
||||
import { CopyToClipboardDirective } from '../../../shared/directives/copy-to-clipboard.directive';
|
||||
import { RelativeTimePipe } from '../../../shared/pipes/relative-time.pipe';
|
||||
import { MetadataGridComponent, MetadataValueComponent } from '../../../shared/components/metadata-grid';
|
||||
|
||||
@@ -31,10 +34,12 @@ import { MetadataGridComponent, MetadataValueComponent } from '../../../shared/c
|
||||
NzSpinModule,
|
||||
NzPopconfirmModule,
|
||||
NzTableModule,
|
||||
NzToolTipModule,
|
||||
RouterLink,
|
||||
RelativeTimePipe,
|
||||
MetadataGridComponent,
|
||||
MetadataValueComponent,
|
||||
CopyToClipboardDirective,
|
||||
],
|
||||
templateUrl: './message-detail.component.html',
|
||||
styleUrl: './message-detail.component.scss'
|
||||
@@ -48,11 +53,13 @@ export class MessageDetailComponent implements OnInit {
|
||||
private settingsService = inject(SettingsService);
|
||||
private keyCacheService = inject(KeyCacheService);
|
||||
private userCacheService = inject(UserCacheService);
|
||||
private clientCacheService = inject(ClientCacheService);
|
||||
|
||||
message = signal<Message | null>(null);
|
||||
resolvedKey = signal<ResolvedKey | null>(null);
|
||||
resolvedChannelOwner = signal<ResolvedUser | null>(null);
|
||||
deliveries = signal<Delivery[]>([]);
|
||||
resolvedClients = signal<Map<string, ResolvedClient>>(new Map());
|
||||
loading = signal(true);
|
||||
deleting = signal(false);
|
||||
loadingDeliveries = signal(false);
|
||||
@@ -107,6 +114,7 @@ export class MessageDetailComponent implements OnInit {
|
||||
next: (response) => {
|
||||
this.deliveries.set(response.deliveries);
|
||||
this.loadingDeliveries.set(false);
|
||||
this.resolveDeliveryClients(response.deliveries);
|
||||
},
|
||||
error: () => {
|
||||
this.loadingDeliveries.set(false);
|
||||
@@ -114,6 +122,27 @@ export class MessageDetailComponent implements OnInit {
|
||||
});
|
||||
}
|
||||
|
||||
private resolveDeliveryClients(deliveries: Delivery[]): void {
|
||||
const uniqueClientIds = [...new Set(deliveries.map(d => d.receiver_client_id))];
|
||||
for (const clientId of uniqueClientIds) {
|
||||
this.clientCacheService.resolveClient(clientId).subscribe({
|
||||
next: (resolved) => {
|
||||
if (resolved) {
|
||||
this.resolvedClients.update(map => {
|
||||
const newMap = new Map(map);
|
||||
newMap.set(clientId, resolved);
|
||||
return newMap;
|
||||
});
|
||||
}
|
||||
}
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
getResolvedClient(clientId: string): ResolvedClient | undefined {
|
||||
return this.resolvedClients().get(clientId);
|
||||
}
|
||||
|
||||
private resolveKey(keyId: string): void {
|
||||
this.keyCacheService.resolveKey(keyId).subscribe({
|
||||
next: (resolved) => this.resolvedKey.set(resolved)
|
||||
|
||||
Reference in New Issue
Block a user