Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b078212a authored by Felipe Leme's avatar Felipe Leme
Browse files

Don't log content of Autofill filter, since it could contain PII. 

Test: manual verification
Fixes: 63680114

Change-Id: I92dcb88df6b577ee2f7a4065e8f9b14ba01ba3fb
parent b12dc429

services/autofill/java/com/android/server/autofill/ui/AutoFillUI.java

+2 −1
Original line number Diff line number Diff line
@@ -163,7 +163,8 @@ public final class AutoFillUI { 
            @Nullable String filterText, @NonNull String packageName,

            @NonNull AutoFillUiCallback callback) {

        if (sDebug) {

            Slog.d(TAG, "showFillUi(): id=" + focusedId + ", filter=" + filterText);

            final int size = filterText == null ? 0 : filterText.length();

            Slog.d(TAG, "showFillUi(): id=" + focusedId + ", filter=" + size + " chars");

        }

        final LogMaker log = (new LogMaker(MetricsProto.MetricsEvent.AUTOFILL_FILL_UI))

                .setPackageName(packageName)

services/autofill/java/com/android/server/autofill/ui/FillUi.java

+4 −1
Original line number Diff line number Diff line
@@ -204,7 +204,10 @@ final class FillUi { 
                return;

            }

            if (count <= 0) {

                if (sDebug) Slog.d(TAG, "No dataset matches filter: " + mFilterText);

                if (sDebug) {

                    final int size = mFilterText == null ? 0 : mFilterText.length();

                    Slog.d(TAG, "No dataset matches filter with " + size + " chars");

                }

                mCallback.requestHideFillUi();

            } else {

                if (updateContentSize()) {