Merge c8fb4c17f1 into 3f7dc10449

README.md

@@ -177,10 +177,6 @@ It's also easy for admins to [add their own custom themes](https://docs.gotosoci
   <img src="https://raw.githubusercontent.com/superseriousbusiness/gotosocial/main/docs/assets/theme-midnight-trip.png"/>
   <figcaption>Midnight trip</figcaption>
 </figure>
-<figure>
-  <img src="https://raw.githubusercontent.com/superseriousbusiness/gotosocial/main/docs/assets/theme-moonlight-hunt.png"/>
-  <figcaption>Moonlight hunt</figcaption>
-</figure>
 <hr/>
 <figure>
   <img src="https://raw.githubusercontent.com/superseriousbusiness/gotosocial/main/docs/assets/theme-rainforest.png"/>

docs/api/swagger.yaml

@@ -950,12 +950,7 @@ definitions:
             with "direct message" visibility.
         properties:
             accounts:
-                description: |-
+                description: Participants in the conversation.
-                    Participants in the conversation.
-
-                    If this is a conversation between no accounts (ie., a self-directed DM),
-                    this will include only the requesting account itself. Otherwise, it will
-                    include every other account in the conversation *except* the requester.
                 items:
                     $ref: '#/definitions/account'
                 type: array
@@ -8955,7 +8950,7 @@ paths:
                     Providing this parameter will cause ScheduledStatus to be returned instead of Status.
                     Must be at least 5 minutes in the future.
 
-                    This feature isn't implemented yet; attemping to set it will return 501 Not Implemented.
+                    This feature isn't implemented yet.
                   in: formData
                   name: scheduled_at
                   type: string
@@ -9016,8 +9011,6 @@ paths:
                     description: not acceptable
                 "500":
                     description: internal server error
-                "501":
-                    description: scheduled_at was set, but this feature is not yet implemented
             security:
                 - OAuth2 Bearer:
                     - write:statuses

docs/configuration/general.md

@@ -80,18 +80,10 @@ host: "localhost"
 # Default: ""
 account-domain: ""
 
-# String. Protocol over which the server is reachable from the outside world.
+# String. Protocol to use for the server. Only change to http for local testing!
-#
+# This should be the protocol part of the URI that your server is actually reachable on. So even if you're
-# ONLY CHANGE THIS TO HTTP FOR LOCAL TESTING! IN 99.99% OF CASES YOU SHOULD NOT CHANGE THIS!
+# running GoToSocial behind a reverse proxy that handles SSL certificates for you, instead of using built-in
-#
+# letsencrypt, it should still be https.
-# This should be the protocol part of the URI that your server is actually reachable on.
-# So even if you're running GoToSocial behind a reverse proxy that handles SSL certificates
-# for you, instead of using built-in letsencrypt, it should still be https, not http.
-#
-# Again, ONLY CHANGE THIS TO HTTP FOR LOCAL TESTING! If you set this to `http`, start your instance,
-# and then later change it to `https`, you will have already broken URI generation for any created
-# users on the instance. You should only touch this setting if you 100% know what you're doing.
-#
 # Options: ["http","https"]
 # Default: "https"
 protocol: "https"

example/config.yaml

@@ -88,18 +88,10 @@ host: "localhost"
 # Default: ""
 account-domain: ""
 
-# String. Protocol over which the server is reachable from the outside world.
+# String. Protocol to use for the server. Only change to http for local testing!
-#
+# This should be the protocol part of the URI that your server is actually reachable on. So even if you're
-# ONLY CHANGE THIS TO HTTP FOR LOCAL TESTING! IN 99.99% OF CASES YOU SHOULD NOT CHANGE THIS!
+# running GoToSocial behind a reverse proxy that handles SSL certificates for you, instead of using built-in
-#
+# letsencrypt, it should still be https.
-# This should be the protocol part of the URI that your server is actually reachable on.
-# So even if you're running GoToSocial behind a reverse proxy that handles SSL certificates
-# for you, instead of using built-in letsencrypt, it should still be https, not http.
-#
-# Again, ONLY CHANGE THIS TO HTTP FOR LOCAL TESTING! If you set this to `http`, start your instance,
-# and then later change it to `https`, you will have already broken URI generation for any created
-# users on the instance. You should only touch this setting if you 100% know what you're doing.
-#
 # Options: ["http","https"]
 # Default: "https"
 protocol: "https"

internal/api/client/statuses/statuscreate.go

@@ -181,7 +181,7 @@
 //			Providing this parameter will cause ScheduledStatus to be returned instead of Status.
 //			Must be at least 5 minutes in the future.
 //
-//			This feature isn't implemented yet; attemping to set it will return 501 Not Implemented.
+//			This feature isn't implemented yet.
 //		type: string
 //		in: formData
 //	-
@@ -254,8 +254,6 @@
 //			description: not acceptable
 //		'500':
 //			description: internal server error
-//		'501':
-//			description: scheduled_at was set, but this feature is not yet implemented
 func (m *Module) StatusCreatePOSTHandler(c *gin.Context) {
 	authed, err := oauth.Authed(c, true, true, true, true)
 	if err != nil {
@@ -288,8 +286,8 @@ func (m *Module) StatusCreatePOSTHandler(c *gin.Context) {
 	// }
 	// form.Status += "\n\nsent from " + user + "'s iphone\n"
 
-	if errWithCode := validateStatusCreateForm(form); errWithCode != nil {
+	if err := validateNormalizeCreateStatus(form); err != nil {
-		apiutil.ErrorHandler(c, errWithCode, m.processor.InstanceGetV1)
+		apiutil.ErrorHandler(c, gtserror.NewErrorBadRequest(err, err.Error()), m.processor.InstanceGetV1)
 		return
 	}
 
@@ -376,61 +374,46 @@ func parseStatusCreateForm(c *gin.Context) (*apimodel.StatusCreateRequest, error
 	return form, nil
 }
 
-// validateStatusCreateForm checks the form for disallowed
+// validateNormalizeCreateStatus checks the form
-// combinations of attachments, overlength inputs, etc.
+// for disallowed combinations of attachments and
+// overlength inputs.
 //
 // Side effect: normalizes the post's language tag.
-func validateStatusCreateForm(form *apimodel.StatusCreateRequest) gtserror.WithCode {
+func validateNormalizeCreateStatus(form *apimodel.StatusCreateRequest) error {
-	var (
+	hasStatus := form.Status != ""
-		chars         = len([]rune(form.Status)) + len([]rune(form.SpoilerText))
+	hasMedia := len(form.MediaIDs) != 0
-		maxChars      = config.GetStatusesMaxChars()
+	hasPoll := form.Poll != nil
-		mediaFiles    = len(form.MediaIDs)
-		maxMediaFiles = config.GetStatusesMediaMaxFiles()
-		hasMedia      = mediaFiles != 0
-		hasPoll       = form.Poll != nil
-	)
 
-	if chars == 0 && !hasMedia && !hasPoll {
+	if !hasStatus && !hasMedia && !hasPoll {
-		// Status must contain *some* kind of content.
+		return errors.New("no status, media, or poll provided")
-		const text = "no status content, content warning, media, or poll provided"
-		return gtserror.NewErrorBadRequest(errors.New(text), text)
 	}
 
-	if chars > maxChars {
+	if hasMedia && hasPoll {
-		text := fmt.Sprintf(
+		return errors.New("can't post media + poll in same status")
-			"status too long, %d characters provided (including content warning) but limit is %d",
-			chars, maxChars,
-		)
-		return gtserror.NewErrorBadRequest(errors.New(text), text)
 	}
 
-	if mediaFiles > maxMediaFiles {
+	maxChars := config.GetStatusesMaxChars()
-		text := fmt.Sprintf(
+	if length := len([]rune(form.Status)) + len([]rune(form.SpoilerText)); length > maxChars {
-			"too many media files attached to status, %d attached but limit is %d",
+		return fmt.Errorf("status too long, %d characters provided (including spoiler/content warning) but limit is %d", length, maxChars)
-			mediaFiles, maxMediaFiles,
+	}
-		)
+
-		return gtserror.NewErrorBadRequest(errors.New(text), text)
+	maxMediaFiles := config.GetStatusesMediaMaxFiles()
+	if len(form.MediaIDs) > maxMediaFiles {
+		return fmt.Errorf("too many media files attached to status, %d attached but limit is %d", len(form.MediaIDs), maxMediaFiles)
 	}
 
 	if form.Poll != nil {
-		if errWithCode := validateStatusPoll(form); errWithCode != nil {
+		if err := validateNormalizeCreatePoll(form); err != nil {
-			return errWithCode
+			return err
 		}
 	}
 
-	if form.ScheduledAt != "" {
-		const text = "scheduled_at is not yet implemented"
-		return gtserror.NewErrorNotImplemented(errors.New(text), text)
-	}
-
-	// Validate + normalize
-	// language tag if provided.
 	if form.Language != "" {
-		lang, err := validate.Language(form.Language)
+		language, err := validate.Language(form.Language)
 		if err != nil {
-			return gtserror.NewErrorBadRequest(err, err.Error())
+			return err
 		}
-		form.Language = lang
+		form.Language = language
 	}
 
 	// Check if the deprecated "federated" field was
@@ -442,36 +425,9 @@ func validateStatusCreateForm(form *apimodel.StatusCreateRequest) gtserror.WithC
 	return nil
 }
 
-func validateStatusPoll(form *apimodel.StatusCreateRequest) gtserror.WithCode {
+func validateNormalizeCreatePoll(form *apimodel.StatusCreateRequest) error {
-	var (
+	maxPollOptions := config.GetStatusesPollMaxOptions()
-		maxPollOptions     = config.GetStatusesPollMaxOptions()
+	maxPollChars := config.GetStatusesPollOptionMaxChars()
-		pollOptions        = len(form.Poll.Options)
-		maxPollOptionChars = config.GetStatusesPollOptionMaxChars()
-	)
-
-	if pollOptions == 0 {
-		const text = "poll with no options"
-		return gtserror.NewErrorBadRequest(errors.New(text), text)
-	}
-
-	if pollOptions > maxPollOptions {
-		text := fmt.Sprintf(
-			"too many poll options provided, %d provided but limit is %d",
-			pollOptions, maxPollOptions,
-		)
-		return gtserror.NewErrorBadRequest(errors.New(text), text)
-	}
-
-	for _, option := range form.Poll.Options {
-		optionChars := len([]rune(option))
-		if optionChars > maxPollOptionChars {
-			text := fmt.Sprintf(
-				"poll option too long, %d characters provided but limit is %d",
-				optionChars, maxPollOptionChars,
-			)
-			return gtserror.NewErrorBadRequest(errors.New(text), text)
-		}
-	}
 
 	// Normalize poll expiry if necessary.
 	// If we parsed this as JSON, expires_in
@@ -484,15 +440,27 @@ func validateStatusPoll(form *apimodel.StatusCreateRequest) gtserror.WithCode {
 		case string:
 			expiresIn, err := strconv.Atoi(e)
 			if err != nil {
-				text := fmt.Sprintf("could not parse expires_in value %s as integer: %v", e, err)
+				return fmt.Errorf("could not parse expires_in value %s as integer: %w", e, err)
-				return gtserror.NewErrorBadRequest(errors.New(text), text)
 			}
 
 			form.Poll.ExpiresIn = expiresIn
 
 		default:
-			text := fmt.Sprintf("could not parse expires_in type %T as integer", ei)
+			return fmt.Errorf("could not parse expires_in type %T as integer", ei)
-			return gtserror.NewErrorBadRequest(errors.New(text), text)
+		}
+	}
+
+	if len(form.Poll.Options) == 0 {
+		return errors.New("poll with no options")
+	}
+
+	if len(form.Poll.Options) > maxPollOptions {
+		return fmt.Errorf("too many poll options provided, %d provided but limit is %d", len(form.Poll.Options), maxPollOptions)
+	}
+
+	for _, p := range form.Poll.Options {
+		if length := len([]rune(p)); length > maxPollChars {
+			return fmt.Errorf("poll option too long, %d characters provided but limit is %d", length, maxPollChars)
 		}
 	}
 

internal/api/client/statuses/statuscreate_test.go

@@ -365,25 +365,6 @@ func (suite *StatusCreateTestSuite) TestPostNewStatusMessedUpIntPolicy() {
 }`, out)
 }
 
-func (suite *StatusCreateTestSuite) TestPostNewScheduledStatus() {
-	out, recorder := suite.postStatus(map[string][]string{
-		"status":       {"this is a brand new status! #helloworld"},
-		"spoiler_text": {"hello hello"},
-		"sensitive":    {"true"},
-		"visibility":   {string(apimodel.VisibilityMutualsOnly)},
-		"scheduled_at": {"2080-10-04T15:32:02.018Z"},
-	}, "")
-
-	// We should have 501 from
-	// our call to the function.
-	suite.Equal(http.StatusNotImplemented, recorder.Code)
-
-	// We should have a helpful error message.
-	suite.Equal(`{
-  "error": "Not Implemented: scheduled_at is not yet implemented"
-}`, out)
-}
-
 func (suite *StatusCreateTestSuite) TestPostNewStatusMarkdown() {
 	out, recorder := suite.postStatus(map[string][]string{
 		"status":       {statusMarkdown},

internal/api/model/conversation.go

@@ -27,10 +27,6 @@ type Conversation struct {
 	// Is the conversation currently marked as unread?
 	Unread bool `json:"unread"`
 	// Participants in the conversation.
-	//
-	// If this is a conversation between no accounts (ie., a self-directed DM),
-	// this will include only the requesting account itself. Otherwise, it will
-	// include every other account in the conversation *except* the requester.
 	Accounts []Account `json:"accounts"`
 	// The last status in the conversation. May be `null`.
 	LastStatus *Status `json:"last_status"`

internal/db/bundb/interaction.go

@@ -302,9 +302,9 @@ func (i *interactionDB) GetInteractionsRequestsForAcct(
 			bun.Ident("interaction_request"),
 		).
 		// Select only interaction requests that
-		// are neither accepted or rejected yet.
+		// are neither accepted or rejected yet,
-		Where("? IS NULL", bun.Ident("accepted_at")).
+		// ie., without an Accept or Reject URI.
-		Where("? IS NULL", bun.Ident("rejected_at"))
+		Where("? IS NULL", bun.Ident("uri"))
 
 	// Select interactions targeting status.
 	if statusID != "" {

internal/db/bundb/migrations/20241005100438_interaction_requests_pending_tweak.go

@@ -1,57 +0,0 @@
-// GoToSocial
-// Copyright (C) GoToSocial Authors admin@gotosocial.org
-// SPDX-License-Identifier: AGPL-3.0-or-later
-//
-// This program is free software: you can redistribute it and/or modify
-// it under the terms of the GNU Affero General Public License as published by
-// the Free Software Foundation, either version 3 of the License, or
-// (at your option) any later version.
-//
-// This program is distributed in the hope that it will be useful,
-// but WITHOUT ANY WARRANTY; without even the implied warranty of
-// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-// GNU Affero General Public License for more details.
-//
-// You should have received a copy of the GNU Affero General Public License
-// along with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-package migrations
-
-import (
-	"context"
-
-	"github.com/uptrace/bun"
-)
-
-func init() {
-	up := func(ctx context.Context, db *bun.DB) error {
-		return db.RunInTx(ctx, nil, func(ctx context.Context, tx bun.Tx) error {
-			for idx, col := range map[string]string{
-				"interaction_requests_accepted_at_idx": "accepted_at",
-				"interaction_requests_rejected_at_idx": "rejected_at",
-			} {
-				if _, err := tx.
-					NewCreateIndex().
-					Table("interaction_requests").
-					Index(idx).
-					Column(col).
-					IfNotExists().
-					Exec(ctx); err != nil {
-					return err
-				}
-			}
-
-			return nil
-		})
-	}
-
-	down := func(ctx context.Context, db *bun.DB) error {
-		return db.RunInTx(ctx, nil, func(ctx context.Context, tx bun.Tx) error {
-			return nil
-		})
-	}
-
-	if err := Migrations.Register(up, down); err != nil {
-		panic(err)
-	}
-}

internal/db/sqlite/driver.go

@@ -112,7 +112,7 @@ func (c *sqliteConn) Close() (err error) {
 	raw := c.connIface.(sqlite3driver.Conn).Raw()
 
 	// see: https://www.sqlite.org/pragma.html#pragma_optimize
-	const onClose = "PRAGMA optimize;"
+	const onClose = "PRAGMA analysis_limit=1000; PRAGMA optimize;"
 	_ = raw.Exec(onClose)
 
 	// Finally, close.

internal/federation/federatingactor.go

@@ -170,6 +170,12 @@ func (f *federatingActor) PostInboxScheme(ctx context.Context, w http.ResponseWr
 	//
 	// Post the activity to the Actor's inbox and trigger side effects.
 	if err := f.sideEffectActor.PostInbox(ctx, inboxID, activity); err != nil {
+		// Check if a function in the federatingDB
+		// has returned an explicit errWithCode for us.
+		if errWithCode, ok := err.(gtserror.WithCode); ok {
+			return false, errWithCode
+		}
+
 		// Check if it's a bad request because the
 		// object or target props weren't populated,
 		// or we failed parsing activity details.
@@ -187,12 +193,6 @@ func (f *federatingActor) PostInboxScheme(ctx context.Context, w http.ResponseWr
 			return false, gtserror.NewErrorBadRequest(errors.New(text), text)
 		}
 
-		// Check if a function in the federatingDB
-		// has returned an explicit errWithCode for us.
-		if errWithCode, ok := err.(gtserror.WithCode); ok {
-			return false, errWithCode
-		}
-
 		// Default: there's been some real error.
 		err := gtserror.Newf("error calling sideEffectActor.PostInbox: %w", err)
 		return false, gtserror.NewErrorInternalError(err)

internal/filter/interaction/interactable.go

@@ -306,7 +306,7 @@ func (f *Filter) StatusBoostable(
 			status.InteractionPolicy.CanAnnounce,
 		)
 
-	// If status has no policy set but it's local,
+	// If status is local and has no policy set,
 	// check against the default policy for this
 	// visibility, as we're interaction-policy aware.
 	case *status.Local:
@@ -318,20 +318,12 @@ func (f *Filter) StatusBoostable(
 			policy.CanAnnounce,
 		)
 
-	// Status is from an instance that does not use
+	// Otherwise, assume the status is from an
-	// or does not care about interaction policies.
+	// instance that does not use / does not care
-	// We can boost it if it's unlisted or public.
+	// about interaction policies, and just return OK.
-	case status.Visibility == gtsmodel.VisibilityPublic ||
-		status.Visibility == gtsmodel.VisibilityUnlocked:
-		return &gtsmodel.PolicyCheckResult{
-			Permission: gtsmodel.PolicyPermissionPermitted,
-		}, nil
-
-	// Not permitted by any of the
-	// above checks, so it's forbidden.
 	default:
 		return &gtsmodel.PolicyCheckResult{
-			Permission: gtsmodel.PolicyPermissionForbidden,
+			Permission: gtsmodel.PolicyPermissionPermitted,
 		}, nil
 	}
 }

internal/gtserror/withcode.go

@@ -191,19 +191,6 @@ func NewErrorGone(original error, helpText ...string) WithCode {
 	}
 }
 
-// NewErrorNotImplemented returns an ErrorWithCode 501 with the given original error and optional help text.
-func NewErrorNotImplemented(original error, helpText ...string) WithCode {
-	safe := http.StatusText(http.StatusNotImplemented)
-	if helpText != nil {
-		safe = safe + ": " + strings.Join(helpText, ": ")
-	}
-	return withCode{
-		original: original,
-		safe:     errors.New(safe),
-		code:     http.StatusNotImplemented,
-	}
-}
-
 // NewErrorClientClosedRequest returns an ErrorWithCode 499 with the given original error.
 // This error type should only be used when an http caller has already hung up their request.
 // See: https://en.wikipedia.org/wiki/List_of_HTTP_status_codes#nginx

internal/processing/common/status.go

@@ -247,12 +247,6 @@ func (p *Processor) GetVisibleAPIStatuses(
 			continue
 		}
 
-		if apiStatus == nil {
-			// Status was
-			// filtered out.
-			continue
-		}
-
 		// Append converted status to return slice.
 		apiStatuses = append(apiStatuses, *apiStatus)
 	}

internal/typeutils/internaltofrontend.go

@@ -1834,23 +1834,46 @@ func (c *Converter) NotificationToAPINotification(
 func (c *Converter) ConversationToAPIConversation(
 	ctx context.Context,
 	conversation *gtsmodel.Conversation,
-	requester *gtsmodel.Account,
+	requestingAccount *gtsmodel.Account,
 	filters []*gtsmodel.Filter,
 	mutes *usermute.CompiledUserMuteList,
 ) (*apimodel.Conversation, error) {
 	apiConversation := &apimodel.Conversation{
-		ID:     conversation.ID,
+		ID:       conversation.ID,
-		Unread: !*conversation.Read,
+		Unread:   !*conversation.Read,
+		Accounts: []apimodel.Account{},
+	}
+	for _, account := range conversation.OtherAccounts {
+		var apiAccount *apimodel.Account
+		blocked, err := c.state.DB.IsEitherBlocked(ctx, requestingAccount.ID, account.ID)
+		if err != nil {
+			return nil, gtserror.Newf(
+				"DB error checking blocks between accounts %s and %s: %w",
+				requestingAccount.ID,
+				account.ID,
+				err,
+			)
+		}
+		if blocked || account.IsSuspended() {
+			apiAccount, err = c.AccountToAPIAccountBlocked(ctx, account)
+		} else {
+			apiAccount, err = c.AccountToAPIAccountPublic(ctx, account)
+		}
+		if err != nil {
+			return nil, gtserror.Newf(
+				"error converting account %s to API representation: %w",
+				account.ID,
+				err,
+			)
+		}
+		apiConversation.Accounts = append(apiConversation.Accounts, *apiAccount)
 	}
-
-	// Populate most recent status in convo;
-	// can be nil if this status is filtered.
 	if conversation.LastStatus != nil {
 		var err error
 		apiConversation.LastStatus, err = c.StatusToAPIStatus(
 			ctx,
 			conversation.LastStatus,
-			requester,
+			requestingAccount,
 			statusfilter.FilterContextNotifications,
 			filters,
 			mutes,
@@ -1864,60 +1887,6 @@ func (c *Converter) ConversationToAPIConversation(
 		}
 	}
 
-	// If no other accounts are involved in this convo,
-	// just include the requesting account and return.
-	//
-	// See: https://github.com/superseriousbusiness/gotosocial/issues/3385#issuecomment-2394033477
-	otherAcctsLen := len(conversation.OtherAccounts)
-	if otherAcctsLen == 0 {
-		apiAcct, err := c.AccountToAPIAccountPublic(ctx, requester)
-		if err != nil {
-			err := gtserror.Newf(
-				"error converting account %s to API representation: %w",
-				requester.ID, err,
-			)
-			return nil, err
-		}
-
-		apiConversation.Accounts = []apimodel.Account{*apiAcct}
-		return apiConversation, nil
-	}
-
-	// Other accounts are involved in the
-	// convo. Convert each to API model.
-	apiConversation.Accounts = make([]apimodel.Account, otherAcctsLen)
-	for i, account := range conversation.OtherAccounts {
-		blocked, err := c.state.DB.IsEitherBlocked(ctx,
-			requester.ID, account.ID,
-		)
-		if err != nil {
-			err := gtserror.Newf(
-				"db error checking blocks between accounts %s and %s: %w",
-				requester.ID, account.ID, err,
-			)
-			return nil, err
-		}
-
-		// API account model varies depending
-		// on status of conversation participant.
-		var apiAcct *apimodel.Account
-		if blocked || account.IsSuspended() {
-			apiAcct, err = c.AccountToAPIAccountBlocked(ctx, account)
-		} else {
-			apiAcct, err = c.AccountToAPIAccountPublic(ctx, account)
-		}
-
-		if err != nil {
-			err := gtserror.Newf(
-				"error converting account %s to API representation: %w",
-				account.ID, err,
-			)
-			return nil, err
-		}
-
-		apiConversation.Accounts[i] = *apiAcct
-	}
-
 	return apiConversation, nil
 }
 
@@ -2713,7 +2682,7 @@ func (c *Converter) InteractionReqToAPIInteractionReq(
 	}
 
 	var reply *apimodel.Status
-	if req.InteractionType == gtsmodel.InteractionReply && req.Reply != nil {
+	if req.InteractionType == gtsmodel.InteractionReply {
 		reply, err = c.statusToAPIStatus(
 			ctx,
 			req.Reply,

internal/typeutils/internaltofrontend_test.go

@@ -3358,321 +3358,6 @@ func (suite *InternalToFrontendTestSuite) TestIntReqToAPI() {
 }`, string(b))
 }
 
-func (suite *InternalToFrontendTestSuite) TestConversationToAPISelfConvo() {
-	var (
-		ctx                                       = context.Background()
-		requester                                 = suite.testAccounts["local_account_1"]
-		lastStatus                                = suite.testStatuses["local_account_1_status_1"]
-		filters    []*gtsmodel.Filter             = nil
-		mutes      *usermute.CompiledUserMuteList = nil
-	)
-
-	convo := &gtsmodel.Conversation{
-		ID:            "01J9C6K86PKZ5GY5WXV94DGH6R",
-		CreatedAt:     testrig.TimeMustParse("2022-06-10T15:22:08Z"),
-		UpdatedAt:     testrig.TimeMustParse("2022-06-10T15:22:08Z"),
-		AccountID:     requester.ID,
-		Account:       requester,
-		OtherAccounts: nil,
-		LastStatus:    lastStatus,
-		Read:          util.Ptr(true),
-	}
-
-	apiConvo, err := suite.typeconverter.ConversationToAPIConversation(
-		ctx,
-		convo,
-		requester,
-		filters,
-		mutes,
-	)
-	if err != nil {
-		suite.FailNow(err.Error())
-	}
-
-	b, err := json.MarshalIndent(apiConvo, "", "  ")
-	if err != nil {
-		suite.FailNow(err.Error())
-	}
-
-	// No other accounts involved, so we should only
-	// have our own account in the "accounts" field.
-	suite.Equal(`{
-  "id": "01J9C6K86PKZ5GY5WXV94DGH6R",
-  "unread": false,
-  "accounts": [
-    {
-      "id": "01F8MH1H7YV1Z7D2C8K2730QBF",
-      "username": "the_mighty_zork",
-      "acct": "the_mighty_zork",
-      "display_name": "original zork (he/they)",
-      "locked": false,
-      "discoverable": true,
-      "bot": false,
-      "created_at": "2022-05-20T11:09:18.000Z",
-      "note": "\u003cp\u003ehey yo this is my profile!\u003c/p\u003e",
-      "url": "http://localhost:8080/@the_mighty_zork",
-      "avatar": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/avatar/original/01F8MH58A357CV5K7R7TJMSH6S.jpg",
-      "avatar_static": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/avatar/small/01F8MH58A357CV5K7R7TJMSH6S.webp",
-      "avatar_description": "a green goblin looking nasty",
-      "header": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/header/original/01PFPMWK2FF0D9WMHEJHR07C3Q.jpg",
-      "header_static": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/header/small/01PFPMWK2FF0D9WMHEJHR07C3Q.webp",
-      "header_description": "A very old-school screenshot of the original team fortress mod for quake",
-      "followers_count": 2,
-      "following_count": 2,
-      "statuses_count": 8,
-      "last_status_at": "2024-01-10T09:24:00.000Z",
-      "emojis": [],
-      "fields": [],
-      "enable_rss": true
-    }
-  ],
-  "last_status": {
-    "id": "01F8MHAMCHF6Y650WCRSCP4WMY",
-    "created_at": "2021-10-20T10:40:37.000Z",
-    "in_reply_to_id": null,
-    "in_reply_to_account_id": null,
-    "sensitive": true,
-    "spoiler_text": "introduction post",
-    "visibility": "public",
-    "language": "en",
-    "uri": "http://localhost:8080/users/the_mighty_zork/statuses/01F8MHAMCHF6Y650WCRSCP4WMY",
-    "url": "http://localhost:8080/@the_mighty_zork/statuses/01F8MHAMCHF6Y650WCRSCP4WMY",
-    "replies_count": 2,
-    "reblogs_count": 1,
-    "favourites_count": 1,
-    "favourited": false,
-    "reblogged": false,
-    "muted": false,
-    "bookmarked": false,
-    "pinned": false,
-    "content": "hello everyone!",
-    "reblog": null,
-    "application": {
-      "name": "really cool gts application",
-      "website": "https://reallycool.app"
-    },
-    "account": {
-      "id": "01F8MH1H7YV1Z7D2C8K2730QBF",
-      "username": "the_mighty_zork",
-      "acct": "the_mighty_zork",
-      "display_name": "original zork (he/they)",
-      "locked": false,
-      "discoverable": true,
-      "bot": false,
-      "created_at": "2022-05-20T11:09:18.000Z",
-      "note": "\u003cp\u003ehey yo this is my profile!\u003c/p\u003e",
-      "url": "http://localhost:8080/@the_mighty_zork",
-      "avatar": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/avatar/original/01F8MH58A357CV5K7R7TJMSH6S.jpg",
-      "avatar_static": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/avatar/small/01F8MH58A357CV5K7R7TJMSH6S.webp",
-      "avatar_description": "a green goblin looking nasty",
-      "header": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/header/original/01PFPMWK2FF0D9WMHEJHR07C3Q.jpg",
-      "header_static": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/header/small/01PFPMWK2FF0D9WMHEJHR07C3Q.webp",
-      "header_description": "A very old-school screenshot of the original team fortress mod for quake",
-      "followers_count": 2,
-      "following_count": 2,
-      "statuses_count": 8,
-      "last_status_at": "2024-01-10T09:24:00.000Z",
-      "emojis": [],
-      "fields": [],
-      "enable_rss": true
-    },
-    "media_attachments": [],
-    "mentions": [],
-    "tags": [],
-    "emojis": [],
-    "card": null,
-    "poll": null,
-    "text": "hello everyone!",
-    "interaction_policy": {
-      "can_favourite": {
-        "always": [
-          "public",
-          "me"
-        ],
-        "with_approval": []
-      },
-      "can_reply": {
-        "always": [
-          "public",
-          "me"
-        ],
-        "with_approval": []
-      },
-      "can_reblog": {
-        "always": [
-          "public",
-          "me"
-        ],
-        "with_approval": []
-      }
-    }
-  }
-}`, string(b))
-}
-
-func (suite *InternalToFrontendTestSuite) TestConversationToAPI() {
-	var (
-		ctx                                       = context.Background()
-		requester                                 = suite.testAccounts["local_account_1"]
-		lastStatus                                = suite.testStatuses["local_account_1_status_1"]
-		filters    []*gtsmodel.Filter             = nil
-		mutes      *usermute.CompiledUserMuteList = nil
-	)
-
-	convo := &gtsmodel.Conversation{
-		ID:        "01J9C6K86PKZ5GY5WXV94DGH6R",
-		CreatedAt: testrig.TimeMustParse("2022-06-10T15:22:08Z"),
-		UpdatedAt: testrig.TimeMustParse("2022-06-10T15:22:08Z"),
-		AccountID: requester.ID,
-		Account:   requester,
-		OtherAccounts: []*gtsmodel.Account{
-			suite.testAccounts["local_account_2"],
-		},
-		LastStatus: lastStatus,
-		Read:       util.Ptr(false),
-	}
-
-	apiConvo, err := suite.typeconverter.ConversationToAPIConversation(
-		ctx,
-		convo,
-		requester,
-		filters,
-		mutes,
-	)
-	if err != nil {
-		suite.FailNow(err.Error())
-	}
-
-	b, err := json.MarshalIndent(apiConvo, "", "  ")
-	if err != nil {
-		suite.FailNow(err.Error())
-	}
-
-	// One other account is involved, so they
-	// should in the "accounts" field and not us.
-	suite.Equal(`{
-  "id": "01J9C6K86PKZ5GY5WXV94DGH6R",
-  "unread": true,
-  "accounts": [
-    {
-      "id": "01F8MH5NBDF2MV7CTC4Q5128HF",
-      "username": "1happyturtle",
-      "acct": "1happyturtle",
-      "display_name": "happy little turtle :3",
-      "locked": true,
-      "discoverable": false,
-      "bot": false,
-      "created_at": "2022-06-04T13:12:00.000Z",
-      "note": "\u003cp\u003ei post about things that concern me\u003c/p\u003e",
-      "url": "http://localhost:8080/@1happyturtle",
-      "avatar": "",
-      "avatar_static": "",
-      "header": "http://localhost:8080/assets/default_header.webp",
-      "header_static": "http://localhost:8080/assets/default_header.webp",
-      "followers_count": 1,
-      "following_count": 1,
-      "statuses_count": 8,
-      "last_status_at": "2021-07-28T08:40:37.000Z",
-      "emojis": [],
-      "fields": [
-        {
-          "name": "should you follow me?",
-          "value": "maybe!",
-          "verified_at": null
-        },
-        {
-          "name": "age",
-          "value": "120",
-          "verified_at": null
-        }
-      ],
-      "hide_collections": true
-    }
-  ],
-  "last_status": {
-    "id": "01F8MHAMCHF6Y650WCRSCP4WMY",
-    "created_at": "2021-10-20T10:40:37.000Z",
-    "in_reply_to_id": null,
-    "in_reply_to_account_id": null,
-    "sensitive": true,
-    "spoiler_text": "introduction post",
-    "visibility": "public",
-    "language": "en",
-    "uri": "http://localhost:8080/users/the_mighty_zork/statuses/01F8MHAMCHF6Y650WCRSCP4WMY",
-    "url": "http://localhost:8080/@the_mighty_zork/statuses/01F8MHAMCHF6Y650WCRSCP4WMY",
-    "replies_count": 2,
-    "reblogs_count": 1,
-    "favourites_count": 1,
-    "favourited": false,
-    "reblogged": false,
-    "muted": false,
-    "bookmarked": false,
-    "pinned": false,
-    "content": "hello everyone!",
-    "reblog": null,
-    "application": {
-      "name": "really cool gts application",
-      "website": "https://reallycool.app"
-    },
-    "account": {
-      "id": "01F8MH1H7YV1Z7D2C8K2730QBF",
-      "username": "the_mighty_zork",
-      "acct": "the_mighty_zork",
-      "display_name": "original zork (he/they)",
-      "locked": false,
-      "discoverable": true,
-      "bot": false,
-      "created_at": "2022-05-20T11:09:18.000Z",
-      "note": "\u003cp\u003ehey yo this is my profile!\u003c/p\u003e",
-      "url": "http://localhost:8080/@the_mighty_zork",
-      "avatar": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/avatar/original/01F8MH58A357CV5K7R7TJMSH6S.jpg",
-      "avatar_static": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/avatar/small/01F8MH58A357CV5K7R7TJMSH6S.webp",
-      "avatar_description": "a green goblin looking nasty",
-      "header": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/header/original/01PFPMWK2FF0D9WMHEJHR07C3Q.jpg",
-      "header_static": "http://localhost:8080/fileserver/01F8MH1H7YV1Z7D2C8K2730QBF/header/small/01PFPMWK2FF0D9WMHEJHR07C3Q.webp",
-      "header_description": "A very old-school screenshot of the original team fortress mod for quake",
-      "followers_count": 2,
-      "following_count": 2,
-      "statuses_count": 8,
-      "last_status_at": "2024-01-10T09:24:00.000Z",
-      "emojis": [],
-      "fields": [],
-      "enable_rss": true
-    },
-    "media_attachments": [],
-    "mentions": [],
-    "tags": [],
-    "emojis": [],
-    "card": null,
-    "poll": null,
-    "text": "hello everyone!",
-    "interaction_policy": {
-      "can_favourite": {
-        "always": [
-          "public",
-          "me"
-        ],
-        "with_approval": []
-      },
-      "can_reply": {
-        "always": [
-          "public",
-          "me"
-        ],
-        "with_approval": []
-      },
-      "can_reblog": {
-        "always": [
-          "public",
-          "me"
-        ],
-        "with_approval": []
-      }
-    }
-  }
-}`, string(b))
-}
-
 func TestInternalToFrontendTestSuite(t *testing.T) {
 	suite.Run(t, new(InternalToFrontendTestSuite))
 }

testrig/testmodels.go

@@ -618,7 +618,7 @@ func NewTestAccounts() map[string]*gtsmodel.Account {
 	}
 
 	if diff := len(accountsSorted) - len(preserializedKeys); diff > 0 {
-		keyStrings := make([]string, 0, diff)
+		keyStrings := make([]string, diff)
 		for i := 0; i < diff; i++ {
 			priv, _ := rsa.GenerateKey(rand.Reader, 2048)
 			key, _ := x509.MarshalPKCS8PrivateKey(priv)

web/assets/themes/moonlight-hunt.css

@@ -1,166 +0,0 @@
-/*
-  theme-title: Moonlight Hunt
-  theme-description: Ominous dark blue / black with a tinge of blood red. You may think it all a mere bad dream.
-*/
-
-:root {
-  /* Define our palette */
-  --bleached-bone: #f3e3d4;
-  --void-blue: #0e131f;
-  --outer-space: #06080e;
-  --ghastly-blue: #88bebe;
-  --blood-red: #6c1619;
-  --bright-red: #f61a1ae6;
-  --feral-orange: #f78d17;
-
-  /* Restyle basic colors */
-  --white1: var(--void-blue);
-  --white2: var(--void-blue);
-  --orange2: var(--bright-red);
-  --blue1: var(--ghastly-blue);
-  --blue2: var(--ghastly-blue);
-  --blue3: var(--ghastly-blue);
-
-  /* Basic page styling (background + foreground) */
-  --bg: var(--void-blue);
-  --bg-accent: var(--void-blue);
-  --fg: var(--bleached-bone);
-  --fg-reduced: var(--bleached-bone);
-  --profile-bg: var(--void-blue); 
-
-  /* Buttons */
-  --bloodshot: linear-gradient(
-    var(--blood-red) 0%,
-    var(--feral-orange) 2%,
-    var(--bright-red) 5%,
-    var(--blood-red) 40%,
-    var(--blood-red) 60%,
-    var(--bright-red) 95%,
-    var(--feral-orange) 98%,
-    var(--blood-red) 100%
-  );
-  --button-bg: var(--bloodshot);
-  --button-fg: var(--bleached-bone);
-
-  /* Statuses */
-  --status-bg: var(--void-blue);
-  --status-focus-bg: var(--void-blue);
-
-  /* Used around statuses + other items */
-  --ghastly-border: 0.1rem solid var(--ghastly-blue);
-  --boxshadow-border: var(--ghastly-border);
-}
-
-/* Main page background */
-body {
-  background: linear-gradient(
-    90deg,
-    var(--blood-red),
-    black 20%,
-    black 80%,
-    var(--blood-red)
-  );
-}
-
-/* Scroll bar */
-html, body {
-  scrollbar-color: var(--bright-red) var(--outer-space);
-  text-shadow: 1px 1px var(--blood-red);
-}
-
-/* Instance title color */
-.page-header a h1 {
-  color: var(--bleached-bone);
-}
-
-.profile .profile-header {
-  border: var(--ghastly-border);
-}
-
-.col-header {
-  border: var(--ghastly-border);
-  background: var(--outer-space);
-}
-
-.profile .about-user .col-header {
-  background: var(--void-blue);
-  border-bottom: none;
-  margin-bottom: 0;
-}
-
-/* Fiddle around with borders on about sections */
-.profile .about-user .fields,
-.profile .about-user .bio,
-.profile .about-user .accountstats {
-  border-left: var(--ghastly-border);
-  border-right: var(--ghastly-border);
-}
-.profile .about-user .accountstats {
-  border-bottom: var(--ghastly-border);
-  background: var(--outer-space);
-}
-
-/* Role and bot badge backgrounds */
-.profile .profile-header .basic-info .namerole .role,
-.profile .profile-header .basic-info .namerole .bot-username-wrapper .bot-legend-wrapper {
-  background: var(--outer-space);
-}
-
-/* Status media */
-.status .media .media-wrapper {
-  border: var(--ghastly-border);
-}
-.status .media .media-wrapper details .unknown-attachment .placeholder {
-  color: var(--bleached-bone);
-}
-.status .media .media-wrapper details video.plyr-video {
-  background: var(--outer-space);
-}
-
-/* Status polls */
-.status .text .poll {
-  background-color: var(--outer-space);
-  border: var(--ghastly-border);
-}
-.status .text .poll .poll-info {
-  background-color: var(--void-blue);
-}
-
-/* Code snippets */
-pre, pre[class*="language-"],
-code, code[class*="language-"] {
-  background-color: var(--outer-space);
-  color: var(--bleached-bone);
-}
-
-/* Block quotes */
-blockquote {
-  background-color: var(--outer-space);
-  color: var(--bleached-bone);
-}
-
-/* Status info bars */
-.status .status-info,
-.status.expanded .status-info {
-  color: var(--ghastly-blue);
-  border-top: 0.1rem dotted var(--ghastly-blue);
-  background: var(--outer-space);
-}
-
-/* Make show more/less buttons more legible */
-.status .button {
-  border: 1px solid var(--feral-orange);
-}
-.status .button:hover {
-  border: 1px solid var(--bleached-bone);
-  background: var(--bloodshot);
-}
-
-/* Back + next links */
-.profile .statuses .backnextlinks a {
-  color: var(--bleached-bone);
-}
-
-.page-footer nav ul li a {
-  color: var(--bleached-bone);
-}