Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

compare field causing daprd panic #4083

Closed
ls-2018 opened this issue Jan 4, 2022 · 0 comments
Closed

compare field causing daprd panic #4083

ls-2018 opened this issue Jan 4, 2022 · 0 comments
Assignees
Labels
kind/bug Something isn't working
Milestone

Comments

@ls-2018
Copy link
Member

ls-2018 commented Jan 4, 2022

In what area(s)?
/area runtime

What version of Dapr?
1.5.0

Expected Behavior
create reminder success

Actual Behavior
dapr panic

Steps to Reproduce the Problem

client.py

data = {
    'Period': "5s",  # 0h30m0s
    'TTL': "30m",  # R5/PT30M  0h30m0s
    'DueTime': "30h",  # R5/PT30M  0h30m0s
    'Data': {},
}
res = requests.post('http://localhost:3500/v1.0/actors/actorType-a/actorId-a/reminders/demo', json.dumps(data))
print(res.text)

bug: /pkg/actors/actors: reminderRequiresUpdate

	data := `{
	"data":{
		"a":"b"
	}
}`
	var a = actors.Reminder{}
	json.Unmarshal([]byte(data), &a)
	var b = actors.CreateReminderRequest{}
	json.Unmarshal([]byte(data), &b)
	//panic: runtime error: comparing uncomparable type map[string]interface {}
	fmt.Println(reflect.DeepEqual(a.Data, b.Data))
	fmt.Println(a.Data == b.Data)

image

@ls-2018 ls-2018 added the kind/bug Something isn't working label Jan 4, 2022
@artursouza artursouza added this to the v1.6 milestone Jan 4, 2022
@ls-2018 ls-2018 closed this as completed Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants