[native] introduce generic for unexpected response handler
ClosedPublic
Actions

Authored by varun on Aug 22 2022, 9:09 PM.

Details

Reviewers

tomek
• karol
atul

Commits

rCOMMe46754ceabfd: [native] introduce generic for unexpected response handler

Summary

Since this function just logs the unexpected message and returns a gRPC Status, we can make it generic

Depends on D4904

Test Plan

cargo build (very small refactor)

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Aug 22 2022, 9:09 PM

Herald added subscribers: • abosh, ashoat. · View Herald TranscriptAug 22 2022, 9:09 PM

varun added a child revision: D4906: [native] helper functions for PAKE login workflow.Aug 22 2022, 9:10 PM

Harbormaster completed remote builds in B11523: Diff 15845.Aug 22 2022, 9:19 PM

varun requested review of this revision.Aug 22 2022, 9:19 PM

tomek accepted this revision.Aug 23 2022, 7:37 AM

This revision is now accepted and ready to land.Aug 23 2022, 7:37 AM

Closed by commit rCOMMe46754ceabfd: [native] introduce generic for unexpected response handler (authored by varun). · Explain WhyAug 23 2022, 8:45 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMe46754ceabfd: [native] introduce generic for unexpected response handler.

Revision Contents
Changeset List

Path

Size

native/

cpp/

CommonCpp/

grpc/

grpc_client/

src/

lib.rs

10 lines

Diff 15845

View Options

native/cpp/CommonCpp/grpc/grpc_client/src/lib.rs

	#!/usr/bin/env bash			#!/usr/bin/env bash

	# Check if shell is using a default BSD or GNU prompt			# Check if shell is using a default BSD, MacOS or GNU prompt
	if [[ ${PS1} != '\s-\v\$'* ]] && \			if [[ ${PS1} != '\s-\v\$'* ]] && \
				[[ ${PS1} != '\h:\W \u\$'* ]] && \
	[[ ${PS1} != '%n@%m %1~ %#'* ]]; then			[[ ${PS1} != '%n@%m %1~ %#'* ]]; then
	# User already has an opinionated PS1, just leave			# User already has an opinionated PS1, just leave
	return 0			return 0
	fi			fi

	if test "$(uname)" = "Darwin" ; then			if test "$(uname)" = "Darwin" ; then
	COMM_CACHE="${XDG_CACHE_HOME:-$HOME/Library/Caches}/app.comm"			COMM_CACHE="${XDG_CACHE_HOME:-$HOME/Library/Caches}/app.comm"
	else			else
	▲ Show 20 Lines • Show All 58 Lines • Show Last 20 Lines

[native] introduce generic for unexpected response handlerClosedPublicActions