[bug] need to skip unexported field

[limpo1989]

Is there an existing issue for this?

• I have searched the existing issues

Current Behavior

create a structInfo but the always inculde unexported field

schema/cache.go

Lines 129 to 139 in 5fca2dc

	func (c *cache) create(t reflect.Type, parentAlias string) *structInfo {
	info := &structInfo{}
	var anonymousInfos []*structInfo
	for i := 0; i < t.NumField(); i++ {
	if f := c.createField(t.Field(i), parentAlias); f != nil {
	info.fields = append(info.fields, f)
	if ft := indirectType(f.typ); ft.Kind() == reflect.Struct && f.isAnonymous {
	anonymousInfos = append(anonymousInfos, c.create(ft, f.canonicalAlias))
	}
	}
	}

Expected Behavior

create a structInfo but the skip unexported field

// create creates a structInfo with meta-data about a struct.
func (c *cache) create(t reflect.Type, parentAlias string) *structInfo {
	info := &structInfo{}
	var anonymousInfos []*structInfo
	for i := 0; i < t.NumField(); i++ {
		field := t.Field(i)
                // !!! skip unexported field
		if !field.IsExported() {
			continue
		}
		if f := c.createField(field, parentAlias); f != nil {
			info.fields = append(info.fields, f)
			if ft := indirectType(f.typ); ft.Kind() == reflect.Struct && f.isAnonymous {
				anonymousInfos = append(anonymousInfos, c.create(ft, f.canonicalAlias))
			}
		}
	}

Steps To Reproduce

No response

Anything else?

No response

[jaitaiwan]

This does seem like it should be expected behaviour but I don't know if there's anywhere in documentation saying that this is the expected behaviour. It seems to me that there should/could be some sort of setting that enables this globally.

[zak905]

I agree with @jaitaiwan, if we want to mimic the behavior of other go standard packages like json, unexported fields are to be ignored. A flag can help maintain backward compatibility, to avoid breaking things. @limpo1989 are you willing to submit a PR?