listmonk/cmd/subscribers.go

661 lines
19 KiB
Go
Raw Normal View History

2018-10-25 21:51:47 +08:00
package main
import (
"context"
"database/sql"
"encoding/json"
2018-10-25 21:51:47 +08:00
"fmt"
"net/http"
"net/url"
2018-10-25 21:51:47 +08:00
"strconv"
"strings"
2020-03-07 23:07:48 +08:00
"github.com/gofrs/uuid"
"github.com/knadh/listmonk/internal/subimporter"
"github.com/knadh/listmonk/models"
2018-10-25 21:51:47 +08:00
"github.com/labstack/echo"
"github.com/lib/pq"
)
const (
dummyUUID = "00000000-0000-0000-0000-000000000000"
)
// subQueryReq is a "catch all" struct for reading various
// subscriber related requests.
type subQueryReq struct {
Query string `json:"query"`
ListIDs pq.Int64Array `json:"list_ids"`
TargetListIDs pq.Int64Array `json:"target_list_ids"`
SubscriberIDs pq.Int64Array `json:"ids"`
Action string `json:"action"`
}
2018-10-25 21:51:47 +08:00
type subsWrap struct {
Results models.Subscribers `json:"results"`
Query string `json:"query"`
Total int `json:"total"`
PerPage int `json:"per_page"`
Page int `json:"page"`
}
// subProfileData represents a subscriber's collated data in JSON
// for export.
type subProfileData struct {
Email string `db:"email" json:"-"`
Profile json.RawMessage `db:"profile" json:"profile,omitempty"`
Subscriptions json.RawMessage `db:"subscriptions" json:"subscriptions,omitempty"`
CampaignViews json.RawMessage `db:"campaign_views" json:"campaign_views,omitempty"`
LinkClicks json.RawMessage `db:"link_clicks" json:"link_clicks,omitempty"`
}
// subOptin contains the data that's passed to the double opt-in e-mail template.
type subOptin struct {
*models.Subscriber
OptinURL string
Lists []models.List
}
var dummySubscriber = models.Subscriber{
Email: "dummy@listmonk.app",
Name: "Dummy Subscriber",
UUID: dummyUUID,
2018-10-25 21:51:47 +08:00
}
// handleGetSubscriber handles the retrieval of a single subscriber by ID.
func handleGetSubscriber(c echo.Context) error {
var (
2018-10-29 17:50:49 +08:00
app = c.Get("app").(*App)
2018-10-25 21:51:47 +08:00
id, _ = strconv.Atoi(c.Param("id"))
)
sub, err := getSubscriber(id, app)
2018-10-25 21:51:47 +08:00
if err != nil {
return err
}
2018-10-25 21:51:47 +08:00
return c.JSON(http.StatusOK, okResp{sub})
2018-10-25 21:51:47 +08:00
}
// handleQuerySubscribers handles querying subscribers based on an arbitrary SQL expression.
2018-10-25 21:51:47 +08:00
func handleQuerySubscribers(c echo.Context) error {
var (
app = c.Get("app").(*App)
2020-08-01 21:27:33 +08:00
pg = getPagination(c.QueryParams(), 30, 100)
2018-10-25 21:51:47 +08:00
// Limit the subscribers to a particular list?
listID, _ = strconv.Atoi(c.FormValue("list_id"))
// The "WHERE ?" bit.
query = sanitizeSQLExp(c.FormValue("query"))
out subsWrap
2018-10-25 21:51:47 +08:00
)
listIDs := pq.Int64Array{}
2018-10-25 21:51:47 +08:00
if listID < 0 {
return echo.NewHTTPError(http.StatusBadRequest, "Invalid `list_id`.")
} else if listID > 0 {
listIDs = append(listIDs, int64(listID))
2018-10-25 21:51:47 +08:00
}
// There's an arbitrary query condition from the frontend.
var (
cond = ""
ordBy = "updated_at"
ord = "DESC"
)
2018-10-25 21:51:47 +08:00
if query != "" {
cond = " AND " + query
}
stmt := fmt.Sprintf(app.queries.QuerySubscribers, cond, ordBy, ord)
2020-03-08 01:30:55 +08:00
2018-10-25 21:51:47 +08:00
// Create a readonly transaction to prevent mutations.
tx, err := app.db.BeginTxx(context.Background(), &sql.TxOptions{ReadOnly: true})
2018-10-25 21:51:47 +08:00
if err != nil {
app.log.Printf("error preparing subscriber query: %v", err)
2018-10-25 21:51:47 +08:00
return echo.NewHTTPError(http.StatusInternalServerError,
2020-03-08 01:30:55 +08:00
fmt.Sprintf("Error preparing subscriber query: %v", pqErrMsg(err)))
2018-10-25 21:51:47 +08:00
}
defer tx.Rollback()
2018-10-25 21:51:47 +08:00
// Run the query. stmt is the raw SQL query.
if err := tx.Select(&out.Results, stmt, listIDs, pg.Offset, pg.Limit); err != nil {
2018-10-25 21:51:47 +08:00
return echo.NewHTTPError(http.StatusInternalServerError,
fmt.Sprintf("Error querying subscribers: %v", pqErrMsg(err)))
}
// Lazy load lists for each subscriber.
if err := out.Results.LoadLists(app.queries.GetSubscriberListsLazy); err != nil {
app.log.Printf("error fetching subscriber lists: %v", err)
2018-10-25 21:51:47 +08:00
return echo.NewHTTPError(http.StatusInternalServerError,
fmt.Sprintf("Error fetching subscriber lists: %v", pqErrMsg(err)))
}
out.Query = query
2018-10-25 21:51:47 +08:00
if len(out.Results) == 0 {
out.Results = make(models.Subscribers, 0)
return c.JSON(http.StatusOK, okResp{out})
}
// Meta.
out.Total = out.Results[0].Total
2018-10-25 21:51:47 +08:00
out.Page = pg.Page
out.PerPage = pg.PerPage
return c.JSON(http.StatusOK, okResp{out})
}
// handleCreateSubscriber handles the creation of a new subscriber.
2018-10-25 21:51:47 +08:00
func handleCreateSubscriber(c echo.Context) error {
var (
app = c.Get("app").(*App)
req subimporter.SubReq
)
// Get and validate fields.
if err := c.Bind(&req); err != nil {
return err
2020-03-07 22:49:22 +08:00
}
req.Email = strings.ToLower(strings.TrimSpace(req.Email))
if err := subimporter.ValidateFields(req); err != nil {
2018-10-25 21:51:47 +08:00
return echo.NewHTTPError(http.StatusBadRequest, err.Error())
}
2020-03-07 22:49:22 +08:00
// Insert the subscriber into the DB.
sub, err := insertSubscriber(req, app)
2018-10-25 21:51:47 +08:00
if err != nil {
2020-03-07 22:49:22 +08:00
return err
2018-10-25 21:51:47 +08:00
}
return c.JSON(http.StatusOK, okResp{sub})
2018-10-25 21:51:47 +08:00
}
// handleUpdateSubscriber handles modification of a subscriber.
2018-10-25 21:51:47 +08:00
func handleUpdateSubscriber(c echo.Context) error {
var (
app = c.Get("app").(*App)
id, _ = strconv.ParseInt(c.Param("id"), 10, 64)
req subimporter.SubReq
)
// Get and validate fields.
if err := c.Bind(&req); err != nil {
return err
}
if id < 1 {
return echo.NewHTTPError(http.StatusBadRequest, "Invalid ID.")
}
2020-03-08 15:33:38 +08:00
if req.Email != "" && !subimporter.IsEmail(req.Email) {
2018-10-25 21:51:47 +08:00
return echo.NewHTTPError(http.StatusBadRequest, "Invalid `email`.")
}
2020-03-08 15:33:38 +08:00
if req.Name != "" && !strHasLen(req.Name, 1, stdInputMaxLen) {
2018-10-25 21:51:47 +08:00
return echo.NewHTTPError(http.StatusBadRequest, "Invalid length for `name`.")
}
_, err := app.queries.UpdateSubscriber.Exec(req.ID,
strings.ToLower(strings.TrimSpace(req.Email)),
strings.TrimSpace(req.Name),
2018-10-25 21:51:47 +08:00
req.Status,
req.Attribs,
req.Lists)
if err != nil {
app.log.Printf("error updating subscriber: %v", err)
2018-10-25 21:51:47 +08:00
return echo.NewHTTPError(http.StatusInternalServerError,
2020-03-08 01:30:55 +08:00
fmt.Sprintf("Error updating subscriber: %v", pqErrMsg(err)))
2018-10-25 21:51:47 +08:00
}
// Send a confirmation e-mail (if there are any double opt-in lists).
sub, err := getSubscriber(int(id), app)
if err != nil {
return err
}
_ = sendOptinConfirmation(sub, []int64(req.Lists), app)
return c.JSON(http.StatusOK, okResp{sub})
2018-10-25 21:51:47 +08:00
}
// handleGetSubscriberSendOptin sends an optin confirmation e-mail to a subscriber.
func handleSubscriberSendOptin(c echo.Context) error {
var (
app = c.Get("app").(*App)
id, _ = strconv.Atoi(c.Param("id"))
out models.Subscribers
)
if id < 1 {
return echo.NewHTTPError(http.StatusBadRequest, "Invalid subscriber ID.")
}
// Fetch the subscriber.
err := app.queries.GetSubscriber.Select(&out, id, nil)
if err != nil {
app.log.Printf("error fetching subscriber: %v", err)
return echo.NewHTTPError(http.StatusInternalServerError,
fmt.Sprintf("Error fetching subscriber: %s", pqErrMsg(err)))
}
if len(out) == 0 {
return echo.NewHTTPError(http.StatusBadRequest, "Subscriber not found.")
}
if err := sendOptinConfirmation(out[0], nil, app); err != nil {
return echo.NewHTTPError(http.StatusBadRequest,
"Error sending opt-in e-mail.")
}
return c.JSON(http.StatusOK, okResp{true})
}
2020-08-01 19:15:29 +08:00
// handleBlocklistSubscribers handles the blocklisting of one or more subscribers.
// It takes either an ID in the URI, or a list of IDs in the request body.
2020-08-01 19:15:29 +08:00
func handleBlocklistSubscribers(c echo.Context) error {
2018-10-25 21:51:47 +08:00
var (
app = c.Get("app").(*App)
pID = c.Param("id")
IDs pq.Int64Array
2018-10-25 21:51:47 +08:00
)
// Is it a /:id call?
if pID != "" {
id, _ := strconv.ParseInt(pID, 10, 64)
if id < 1 {
return echo.NewHTTPError(http.StatusBadRequest, "Invalid ID.")
}
IDs = append(IDs, id)
} else {
// Multiple IDs.
var req subQueryReq
if err := c.Bind(&req); err != nil {
return echo.NewHTTPError(http.StatusBadRequest,
fmt.Sprintf("One or more invalid IDs given: %v", err))
}
if len(req.SubscriberIDs) == 0 {
return echo.NewHTTPError(http.StatusBadRequest,
"No IDs given.")
}
IDs = req.SubscriberIDs
2018-10-25 21:51:47 +08:00
}
2020-08-01 19:15:29 +08:00
if _, err := app.queries.BlocklistSubscribers.Exec(IDs); err != nil {
app.log.Printf("error blocklisting subscribers: %v", err)
2018-10-25 21:51:47 +08:00
return echo.NewHTTPError(http.StatusInternalServerError,
2020-08-01 19:15:29 +08:00
fmt.Sprintf("Error blocklisting: %v", err))
2018-10-25 21:51:47 +08:00
}
return c.JSON(http.StatusOK, okResp{true})
}
// handleManageSubscriberLists handles bulk addition or removal of subscribers
// from or to one or more target lists.
// It takes either an ID in the URI, or a list of IDs in the request body.
func handleManageSubscriberLists(c echo.Context) error {
2018-10-25 21:51:47 +08:00
var (
app = c.Get("app").(*App)
pID = c.Param("id")
IDs pq.Int64Array
2018-10-25 21:51:47 +08:00
)
// Is it a /:id call?
if pID != "" {
id, _ := strconv.ParseInt(pID, 10, 64)
if id < 1 {
return echo.NewHTTPError(http.StatusBadRequest, "Invalid ID.")
}
IDs = append(IDs, id)
}
var req subQueryReq
2018-10-25 21:51:47 +08:00
if err := c.Bind(&req); err != nil {
return echo.NewHTTPError(http.StatusBadRequest,
fmt.Sprintf("One or more invalid IDs given: %v", err))
2018-10-25 21:51:47 +08:00
}
if len(req.SubscriberIDs) == 0 {
return echo.NewHTTPError(http.StatusBadRequest,
"No IDs given.")
}
if len(IDs) == 0 {
IDs = req.SubscriberIDs
}
if len(req.TargetListIDs) == 0 {
return echo.NewHTTPError(http.StatusBadRequest, "No lists given.")
2018-10-25 21:51:47 +08:00
}
// Action.
var err error
switch req.Action {
case "add":
_, err = app.queries.AddSubscribersToLists.Exec(IDs, req.TargetListIDs)
case "remove":
_, err = app.queries.DeleteSubscriptions.Exec(IDs, req.TargetListIDs)
case "unsubscribe":
_, err = app.queries.UnsubscribeSubscribersFromLists.Exec(IDs, req.TargetListIDs)
default:
return echo.NewHTTPError(http.StatusBadRequest, "Invalid action.")
2018-10-25 21:51:47 +08:00
}
if err != nil {
app.log.Printf("error updating subscriptions: %v", err)
return echo.NewHTTPError(http.StatusInternalServerError,
fmt.Sprintf("Error processing lists: %v", err))
2018-10-25 21:51:47 +08:00
}
return c.JSON(http.StatusOK, okResp{true})
}
// handleDeleteSubscribers handles subscriber deletion.
// It takes either an ID in the URI, or a list of IDs in the request body.
func handleDeleteSubscribers(c echo.Context) error {
2018-10-25 21:51:47 +08:00
var (
app = c.Get("app").(*App)
pID = c.Param("id")
IDs pq.Int64Array
2018-10-25 21:51:47 +08:00
)
// Is it an /:id call?
if pID != "" {
id, _ := strconv.ParseInt(pID, 10, 64)
if id < 1 {
return echo.NewHTTPError(http.StatusBadRequest, "Invalid ID.")
}
IDs = append(IDs, id)
2018-10-25 21:51:47 +08:00
} else {
// Multiple IDs.
i, err := parseStringIDs(c.Request().URL.Query()["id"])
if err != nil {
return echo.NewHTTPError(http.StatusBadRequest,
fmt.Sprintf("One or more invalid IDs given: %v", err))
}
if len(i) == 0 {
return echo.NewHTTPError(http.StatusBadRequest,
"No IDs given.")
}
IDs = i
2018-10-25 21:51:47 +08:00
}
if _, err := app.queries.DeleteSubscribers.Exec(IDs, nil); err != nil {
app.log.Printf("error deleting subscribers: %v", err)
2018-10-25 21:51:47 +08:00
return echo.NewHTTPError(http.StatusInternalServerError,
2020-03-08 01:30:55 +08:00
fmt.Sprintf("Error deleting subscribers: %v", err))
2018-10-25 21:51:47 +08:00
}
return c.JSON(http.StatusOK, okResp{true})
}
// handleDeleteSubscribersByQuery bulk deletes based on an
// arbitrary SQL expression.
func handleDeleteSubscribersByQuery(c echo.Context) error {
var (
app = c.Get("app").(*App)
req subQueryReq
)
if err := c.Bind(&req); err != nil {
return err
2018-10-25 21:51:47 +08:00
}
err := app.queries.execSubscriberQueryTpl(sanitizeSQLExp(req.Query),
app.queries.DeleteSubscribersByQuery,
req.ListIDs, app.db)
if err != nil {
app.log.Printf("error querying subscribers: %v", err)
return echo.NewHTTPError(http.StatusBadRequest,
fmt.Sprintf("Error: %v", err))
}
return c.JSON(http.StatusOK, okResp{true})
}
2020-08-01 19:15:29 +08:00
// handleBlocklistSubscribersByQuery bulk blocklists subscribers
// based on an arbitrary SQL expression.
2020-08-01 19:15:29 +08:00
func handleBlocklistSubscribersByQuery(c echo.Context) error {
var (
app = c.Get("app").(*App)
req subQueryReq
)
if err := c.Bind(&req); err != nil {
return err
2018-10-25 21:51:47 +08:00
}
err := app.queries.execSubscriberQueryTpl(sanitizeSQLExp(req.Query),
2020-08-01 19:15:29 +08:00
app.queries.BlocklistSubscribersByQuery,
req.ListIDs, app.db)
2018-10-25 21:51:47 +08:00
if err != nil {
2020-08-01 19:15:29 +08:00
app.log.Printf("error blocklisting subscribers: %v", err)
return echo.NewHTTPError(http.StatusBadRequest,
fmt.Sprintf("Error: %v", err))
2018-10-25 21:51:47 +08:00
}
return c.JSON(http.StatusOK, okResp{true})
}
2020-08-01 19:15:29 +08:00
// handleManageSubscriberListsByQuery bulk adds/removes/unsubscribers subscribers
// from one or more lists based on an arbitrary SQL expression.
func handleManageSubscriberListsByQuery(c echo.Context) error {
var (
app = c.Get("app").(*App)
req subQueryReq
)
if err := c.Bind(&req); err != nil {
return err
}
if len(req.TargetListIDs) == 0 {
return echo.NewHTTPError(http.StatusBadRequest, "No lists given.")
}
// Action.
var stmt string
switch req.Action {
case "add":
stmt = app.queries.AddSubscribersToListsByQuery
case "remove":
stmt = app.queries.DeleteSubscriptionsByQuery
case "unsubscribe":
stmt = app.queries.UnsubscribeSubscribersFromListsByQuery
default:
return echo.NewHTTPError(http.StatusBadRequest, "Invalid action.")
}
err := app.queries.execSubscriberQueryTpl(sanitizeSQLExp(req.Query),
stmt, req.ListIDs, app.db, req.TargetListIDs)
2018-10-25 21:51:47 +08:00
if err != nil {
app.log.Printf("error updating subscriptions: %v", err)
return echo.NewHTTPError(http.StatusBadRequest,
fmt.Sprintf("Error: %v", err))
2018-10-25 21:51:47 +08:00
}
return c.JSON(http.StatusOK, okResp{true})
2018-10-25 21:51:47 +08:00
}
// handleExportSubscriberData pulls the subscriber's profile,
// list subscriptions, campaign views and clicks and produces
// a JSON report. This is a privacy feature and depends on the
// configuration in app.Constants.Privacy.
func handleExportSubscriberData(c echo.Context) error {
var (
app = c.Get("app").(*App)
pID = c.Param("id")
)
id, _ := strconv.ParseInt(pID, 10, 64)
if id < 1 {
return echo.NewHTTPError(http.StatusBadRequest, "Invalid ID.")
}
// Get the subscriber's data. A single query that gets the profile,
// list subscriptions, campaign views, and link clicks. Names of
// private lists are replaced with "Private list".
_, b, err := exportSubscriberData(id, "", app.constants.Privacy.Exportable, app)
if err != nil {
app.log.Printf("error exporting subscriber data: %s", err)
return echo.NewHTTPError(http.StatusBadRequest,
"Error exporting subscriber data.")
}
c.Response().Header().Set("Cache-Control", "no-cache")
c.Response().Header().Set("Content-Disposition", `attachment; filename="data.json"`)
return c.Blob(http.StatusOK, "application/json", b)
}
2020-03-07 22:49:22 +08:00
// insertSubscriber inserts a subscriber and returns the ID.
func insertSubscriber(req subimporter.SubReq, app *App) (models.Subscriber, error) {
2020-03-07 23:07:48 +08:00
uu, err := uuid.NewV4()
if err != nil {
return req.Subscriber, err
2020-03-07 23:07:48 +08:00
}
req.UUID = uu.String()
err = app.queries.InsertSubscriber.Get(&req.ID,
2020-03-07 22:49:22 +08:00
req.UUID,
req.Email,
strings.TrimSpace(req.Name),
req.Status,
req.Attribs,
req.Lists,
req.ListUUIDs)
if err != nil {
if pqErr, ok := err.(*pq.Error); ok && pqErr.Constraint == "subscribers_email_key" {
return req.Subscriber, echo.NewHTTPError(http.StatusBadRequest, "The e-mail already exists.")
2020-03-07 22:49:22 +08:00
}
2020-03-08 01:30:55 +08:00
app.log.Printf("error inserting subscriber: %v", err)
return req.Subscriber, echo.NewHTTPError(http.StatusInternalServerError,
2020-03-08 01:30:55 +08:00
fmt.Sprintf("Error inserting subscriber: %v", err))
2020-03-07 22:49:22 +08:00
}
// Fetch the subscriber's full data.
sub, err := getSubscriber(req.ID, app)
if err != nil {
return sub, err
}
// Send a confirmation e-mail (if there are any double opt-in lists).
_ = sendOptinConfirmation(sub, []int64(req.Lists), app)
return sub, nil
}
// getSubscriber gets a single subscriber by ID.
func getSubscriber(id int, app *App) (models.Subscriber, error) {
var (
out models.Subscribers
)
if id < 1 {
return models.Subscriber{}, echo.NewHTTPError(http.StatusBadRequest, "Invalid subscriber ID.")
}
if err := app.queries.GetSubscriber.Select(&out, id, nil); err != nil {
app.log.Printf("error fetching subscriber: %v", err)
return models.Subscriber{}, echo.NewHTTPError(http.StatusInternalServerError,
fmt.Sprintf("Error fetching subscriber: %s", pqErrMsg(err)))
}
if len(out) == 0 {
return models.Subscriber{}, echo.NewHTTPError(http.StatusBadRequest, "Subscriber not found.")
}
if err := out.LoadLists(app.queries.GetSubscriberListsLazy); err != nil {
app.log.Printf("error loading subscriber lists: %v", err)
return models.Subscriber{}, echo.NewHTTPError(http.StatusInternalServerError,
"Error loading subscriber lists.")
}
return out[0], nil
2020-03-07 22:49:22 +08:00
}
// exportSubscriberData collates the data of a subscriber including profile,
// subscriptions, campaign_views, link_clicks (if they're enabled in the config)
// and returns a formatted, indented JSON payload. Either takes a numeric id
// and an empty subUUID or takes 0 and a string subUUID.
func exportSubscriberData(id int64, subUUID string, exportables map[string]bool, app *App) (subProfileData, []byte, error) {
// Get the subscriber's data. A single query that gets the profile,
// list subscriptions, campaign views, and link clicks. Names of
// private lists are replaced with "Private list".
var (
data subProfileData
uu interface{}
)
// UUID should be a valid value or a nil.
if subUUID != "" {
uu = subUUID
}
if err := app.queries.ExportSubscriberData.Get(&data, id, uu); err != nil {
app.log.Printf("error fetching subscriber export data: %v", err)
return data, nil, err
}
// Filter out the non-exportable items.
if _, ok := exportables["profile"]; !ok {
data.Profile = nil
}
if _, ok := exportables["subscriptions"]; !ok {
data.Subscriptions = nil
}
if _, ok := exportables["campaign_views"]; !ok {
data.CampaignViews = nil
}
if _, ok := exportables["link_clicks"]; !ok {
data.LinkClicks = nil
}
// Marshal the data into an indented payload.
b, err := json.MarshalIndent(data, "", " ")
if err != nil {
app.log.Printf("error marshalling subscriber export data: %v", err)
return data, nil, err
}
return data, b, nil
}
// sendOptinConfirmation sends a double opt-in confirmation e-mail to a subscriber
// if at least one of the given listIDs is set to optin=double
func sendOptinConfirmation(sub models.Subscriber, listIDs []int64, app *App) error {
var lists []models.List
// Fetch double opt-in lists from the given list IDs.
// Get the list of subscription lists where the subscriber hasn't confirmed.
if err := app.queries.GetSubscriberLists.Select(&lists, sub.ID, nil,
pq.Int64Array(listIDs), nil, models.SubscriptionStatusUnconfirmed, models.ListOptinDouble); err != nil {
app.log.Printf("error fetching lists for opt-in: %s", pqErrMsg(err))
return err
}
// None.
if len(lists) == 0 {
return nil
}
var (
out = subOptin{Subscriber: &sub, Lists: lists}
qListIDs = url.Values{}
)
// Construct the opt-in URL with list IDs.
for _, l := range out.Lists {
qListIDs.Add("l", l.UUID)
}
out.OptinURL = fmt.Sprintf(app.constants.OptinURL, sub.UUID, qListIDs.Encode())
// Send the e-mail.
if err := app.sendNotification([]string{sub.Email},
"Confirm subscription", notifSubscriberOptin, out); err != nil {
app.log.Printf("error e-mailing subscriber profile: %s", err)
return err
}
return nil
}
// sanitizeSQLExp does basic sanitisation on arbitrary
// SQL query expressions coming from the frontend.
func sanitizeSQLExp(q string) string {
if len(q) == 0 {
return ""
}
q = strings.TrimSpace(q)
// Remove semicolon suffix.
if q[len(q)-1] == ';' {
q = q[:len(q)-1]
}
return q
}