Merge branch 'cycles-pipeline' into 'develop'
Recompilation speedup: switch to runtime deps in ActivityPub.Pipeline See merge request pleroma/pleroma!3427
This commit is contained in:
		
						commit
						018dc12b4c
					
				
					 1 changed files with 13 additions and 13 deletions
				
			
		| 
						 | 
				
			
			@ -15,19 +15,19 @@ defmodule Pleroma.Web.ActivityPub.Pipeline do
 | 
			
		|||
  alias Pleroma.Web.ActivityPub.Visibility
 | 
			
		||||
  alias Pleroma.Web.Federator
 | 
			
		||||
 | 
			
		||||
  @side_effects Config.get([:pipeline, :side_effects], SideEffects)
 | 
			
		||||
  @federator Config.get([:pipeline, :federator], Federator)
 | 
			
		||||
  @object_validator Config.get([:pipeline, :object_validator], ObjectValidator)
 | 
			
		||||
  @mrf Config.get([:pipeline, :mrf], MRF)
 | 
			
		||||
  @activity_pub Config.get([:pipeline, :activity_pub], ActivityPub)
 | 
			
		||||
  @config Config.get([:pipeline, :config], Config)
 | 
			
		||||
  defp side_effects, do: Config.get([:pipeline, :side_effects], SideEffects)
 | 
			
		||||
  defp federator, do: Config.get([:pipeline, :federator], Federator)
 | 
			
		||||
  defp object_validator, do: Config.get([:pipeline, :object_validator], ObjectValidator)
 | 
			
		||||
  defp mrf, do: Config.get([:pipeline, :mrf], MRF)
 | 
			
		||||
  defp activity_pub, do: Config.get([:pipeline, :activity_pub], ActivityPub)
 | 
			
		||||
  defp config, do: Config.get([:pipeline, :config], Config)
 | 
			
		||||
 | 
			
		||||
  @spec common_pipeline(map(), keyword()) ::
 | 
			
		||||
          {:ok, Activity.t() | Object.t(), keyword()} | {:error, any()}
 | 
			
		||||
  def common_pipeline(object, meta) do
 | 
			
		||||
    case Repo.transaction(fn -> do_common_pipeline(object, meta) end, Utils.query_timeout()) do
 | 
			
		||||
      {:ok, {:ok, activity, meta}} ->
 | 
			
		||||
        @side_effects.handle_after_transaction(meta)
 | 
			
		||||
        side_effects().handle_after_transaction(meta)
 | 
			
		||||
        {:ok, activity, meta}
 | 
			
		||||
 | 
			
		||||
      {:ok, value} ->
 | 
			
		||||
| 
						 | 
				
			
			@ -44,10 +44,10 @@ def common_pipeline(object, meta) do
 | 
			
		|||
  def do_common_pipeline(%{__struct__: _}, _meta), do: {:error, :is_struct}
 | 
			
		||||
 | 
			
		||||
  def do_common_pipeline(message, meta) do
 | 
			
		||||
    with {_, {:ok, message, meta}} <- {:validate, @object_validator.validate(message, meta)},
 | 
			
		||||
         {_, {:ok, message, meta}} <- {:mrf, @mrf.pipeline_filter(message, meta)},
 | 
			
		||||
         {_, {:ok, message, meta}} <- {:persist, @activity_pub.persist(message, meta)},
 | 
			
		||||
         {_, {:ok, message, meta}} <- {:side_effects, @side_effects.handle(message, meta)},
 | 
			
		||||
    with {_, {:ok, message, meta}} <- {:validate, object_validator().validate(message, meta)},
 | 
			
		||||
         {_, {:ok, message, meta}} <- {:mrf, mrf().pipeline_filter(message, meta)},
 | 
			
		||||
         {_, {:ok, message, meta}} <- {:persist, activity_pub().persist(message, meta)},
 | 
			
		||||
         {_, {:ok, message, meta}} <- {:side_effects, side_effects().handle(message, meta)},
 | 
			
		||||
         {_, {:ok, _}} <- {:federation, maybe_federate(message, meta)} do
 | 
			
		||||
      {:ok, message, meta}
 | 
			
		||||
    else
 | 
			
		||||
| 
						 | 
				
			
			@ -60,7 +60,7 @@ defp maybe_federate(%Object{}, _), do: {:ok, :not_federated}
 | 
			
		|||
 | 
			
		||||
  defp maybe_federate(%Activity{} = activity, meta) do
 | 
			
		||||
    with {:ok, local} <- Keyword.fetch(meta, :local) do
 | 
			
		||||
      do_not_federate = meta[:do_not_federate] || !@config.get([:instance, :federating])
 | 
			
		||||
      do_not_federate = meta[:do_not_federate] || !config().get([:instance, :federating])
 | 
			
		||||
 | 
			
		||||
      if !do_not_federate and local and not Visibility.is_local_public?(activity) do
 | 
			
		||||
        activity =
 | 
			
		||||
| 
						 | 
				
			
			@ -70,7 +70,7 @@ defp maybe_federate(%Activity{} = activity, meta) do
 | 
			
		|||
            activity
 | 
			
		||||
          end
 | 
			
		||||
 | 
			
		||||
        @federator.publish(activity)
 | 
			
		||||
        federator().publish(activity)
 | 
			
		||||
        {:ok, :federated}
 | 
			
		||||
      else
 | 
			
		||||
        {:ok, :not_federated}
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue