Problem running backend

I'm getting an ImportError: cannot import name 'FastAPICache' from 'fastapi_cache' when trying to run the backend. I had to manually install the aioredis and fastapi_cache dependencies. Any ideas, please?

Yes, they are dependencies indeed. I will update the requirements file.

Thanks, however, the ImportError does not go away. Sorry, I should have stated that in my initial posting.

I ll get back to you this evening and try to figure it out.

Can you just tell me what type of system you are running? Windows, Linux?

Sure: macOS 12.6 and Python 3.10.7 in a virtual env created with python3 -m venv.

Excuse me, did you install https://github.com/long2ice/fastapi-cache? It is called fastapi-cache2 with the 2. I have checked and I used:
pip install fastapi-cache2 aioredis
but maybe you should add
pip install "fastapi-cache2[redis]"
Please let me know if this solves the issue. I haven't been able to develop on a Mac, I only tested on Windows (11) and the latest Ubuntu.

Great, glad I could help!


Adding Hebrew - crowdin

TNX

@tzagim happy translatung - may wana close this issue ;)

@6543 And finish....

@helloworld1
please pay attention
Google decided to use in Android IW code for Hebrew and not HE code.

That's very weird. I can move the files to values-iw or keep both iw and he.
I will use emulator to test the locale later.

this is strange ... will have to change config in crowdin so export will follow new name ... waiting for your test result :)

That's indeed very weird. I set the emulator locale to Hebrew and locale he-IL. It still displays English on emulator.
$ adb shell getprop | grep locale
[persist.sys.locale]: [he-IL]
I moved values-he to  values-iw and it displays Hebrew correctly.

It is also possible to make a shortcut from values-he to values-iw.

Google has a lint check and ask devs to use values-iw insteads of he. It gives the reason there:
https://android.googlesource.com/platform/tools/base/+/master/lint/libs/lint-checks/src/main/java/com/android/tools/lint/checks/LocaleFolderDetector.java#67
So I think we are good on just moving values-he to values-iw

@6543 Would you help to move Crowdin translations to values-iw? I can merge a PR later. Thanks!

Yes sure just give me some time :)

@6543 Would you help to move Crowdin translations to values-iw? I can merge a PR later. Thanks!
#155

I could not find the option to rename leter - I would have to add a "custom" language and call it Hebrew


Calling Close in another thread crashes

When Start() is called in one thread, therefore blocking it, and Stop() is called in another thread, libfuse.so.2 crashes with a SIGSEGV in fuse_exit().
May be similar to https://github.com/fusepy/fusepy/issues/116

@claunia Yeah. As I understand from FUSE docs fuse_exit and thus Stop() can be used only inside file system operations.
You can use workaround, as mentioned in fusepy/fusepy#116:
using Mono.Unix.Native; // already exists as Mono.Fuse.NETStandard dependency
...
fuseThreadPid = Syscall.getpid();
fs.Start();
...
//in another thread
Syscall.kill(fuseThreadPid, Signum.SIGHUP); //instead fs.Stop ();

I'll add informative exception in Stop() to clarify this moment.
fuse_main that used in Start() doesnt allow any in-process stop from another thread, but it is possible to use more low-level calls to implement this. I think I need to implement it as another method like StartEventLoop(). And with it Stop() will work in expected intuitive manner.


phonegap build

Hi, is there a way to use this plugin with phonegap build? thank you

Hi,
I'm not a phonegap | cordova specialist, I'm a mobile developer.
I have no experience with phonegap build but I guess that this link can help you:
http://docs.build.phonegap.com/en_US/configuring_plugins.md.html
I have always added my plugins with the Cordova Command Line Interface, http://cordova.apache.org/docs/en/5.0.0/guide_cli_index.md.html#The Command-Line Interface
Does this answer your question?

thanks Francesco. I guess I'll keep looking :), but since there is a step to actually add the Fabric plugin to my IDE, it might not be possible with phonegap.


Authentication feature for multiple account same device login

I am currently allowing users to log in/signup by using the fcl API to interact with blocto.
Assume we have users A and B who both use the same device.
User A logs into his account to do some transactions then logs out and leaves the device. User B then tries to log in but when he presses the login button, they see the following.

So now User B can either press confirm and log into user's A account or click the "use Another Account" button to log into his account.
What is mentioned above is very inconvenient especially if a user has multiple accounts and wants to perform transactions on one account but presses "confirm" by mistake and accidentally performs the transactions on a different account.
What I am asking is if there is a way to the following whenever a user presses the login button

Also, if there is already a workaround solution, could someone please guide me on how to overcome this issue?

There is a workaround solution:
After disconnect, click connect again, select "Use another account", and close popup without typing new email.
When clicking "Use another account", the session that store your email will be removed.
Since fcl.js also only has "disconnect" method, I have no idea how to add a method that support "deeply logout".
You can open a new issue discuss with fcl.js here: https://github.com/onflow/fcl-js/


Unable to use --service-account - No valid arguments given

Hi there,
trying to use the --service-account variant but I'm always getting this error msg:
No valid arguments given, use 'gdrive help' to see available commands

command is: gdrive --service-account key.json about
and the key.jsonis in /Users/konstantin/.gdrive
I'm on osx and used brew to install gdrive. Version is 2.1.0. I've quickly looked to the Go code but couldn't find anything why this shouldn't work. Somebody have a clue?

Just tried the version here from github releases page. This one works. Same version.
from github page
~/Downloads/gdrive-osx-x64 version
gdrive: 2.1.0
Golang: go1.10.4
OS/Arch: darwin/amd64

from brew
gdrive version
gdrive: 2.1.0
Golang: go1.13.1
OS/Arch: darwin/amd64

I am still getting this error could you check my info on https://github.com/prasmussen/gdrive/issues/533#issuecomment-784019772

You also need to use the --config param to be able to use --service-account, example:
./gdrive --config <path-to-json-file> --service-account <json-file-name> list


[suggestion] Fire Resistance Effect grants immunity to heat

It just makes sense, and without this, Fire Resistance starts becoming... somewhat obsolete agaisnt lava.

omg I forgot to add this, my bad


data-fetching-astro-glob

New or updated content

A quick example of Astro.glob() on the data fetching page until we have a proper "Here's how to Content with Astro" full story. This is better exposure for an example of creating an index of blog posts, which currently only exists in Reference and is not as accessible.
Now, this page details:

Fetching data from local files to use in a component (Astro.glob)
in addition to what was previously there:
fetch from remote APIs, incl GraphQL, in both Astro and UI Framework components
fetch from a headless CMS

Just a note to say @sarah11918 & I did discuss this and concluded that this didn’t really belong on data “fetching” because that is usually understood to mean “remote” data. We can keep this PR around for reference if it’s useful for a future content-focused guide, but otherwise this can be closed!


Create other types of tiles

It's not yet 100% planned feature, but it's a good idea to consider creating other kind of tiles (.hmp, .qmt, etc).

Talking about DEMs, it's theoretically easy: just use gdal's methods, but I'd like to implement my own way to do it.


assertThatThrownBy doesNotThrowAnyException fail when callable doesnt throw any exception

Summary
In my test the method call should pass if none exception is throwed, but if the ThrowingCallable doesn't throw anything an assertion error happens
Example

@Test
@DisplayName("Should Delete a person by CPF and then save another person with this cpf")
void shouldDeleteAPersonByCPFAndThenSaveAnotherPersonWithThisCPF() {
     assertThatThrownBy(() -> { service.batchOperationByCPF(CPF); }).doesNotThrowAnyException();
}

Am I doing something wrong?
java.lang.AssertionError:
Expecting code to raise a throwable.
	at com.dtb.jpadeletetest.service.PersonServiceIT.shouldDeleteAPersonByCPFAndThenSaveAnotherPersonWithThisCPF(PersonServiceIT.java:38)

assertThatCode worked.


MAINTAINERS: add Laird Connectivity to mcumgr

Laird Connectivity relies on mcumgr as the main
communication solution for our products over BLE and
UART.

Will re-open when we determine an individual who is going to collaborate.

Will re-open when we determine an individual who is going to collaborate.

@rerickson1 this already happened: https://github.com/zephyrproject-rtos/zephyr/pull/39988


importDocumentNodeExternallyFrom generates incorrect import path

Describe the bug
When using importDocumentNodeExternallyFrom: 'near-operation-file' in combination with documentMode: 'external' import paths are not correctly generated if the near-operation-file specifies a custom folder
To Reproduce
Use a config like the following
overwrite: true
schema: 'schema.graphql'
documents: './graphql/**/*.gql.ts'
generates:
  /:
    preset: near-operation-file
    presetConfig:
      extension: .generated.tsx
      baseTypesPath: types.ts
      folder: generated
      cwd: '/'
    plugins:
      - 'typescript-operations'
      - 'typescript-react-apollo'
    config:
      documentMode: 'external'
      importDocumentNodeExternallyFrom: 'near-operation-file'
      withHooks: true
      withHOC: false
      reactApolloVersion: 3

It looks like it is hard-coded to import from the same folder here:
https://github.com/dotansimha/graphql-code-generator/blob/cc8ca46b120bd30620cee9d2897c93fa4db51c1a/packages/plugins/other/visitor-plugin-common/src/client-side-base-visitor.ts#L306

For anyone else running into this, we are working around it with find and sed as follows`
Note this only works if your Operations files are one folder above your generated folder
QUERIES_GENERATED_FOLDER=./graphql/queries/generated
MUTATIONS_GENERATED_FOLDER=./graphql/mutations/generated

find $QUERIES_GENERATED_FOLDER -type f -name "*.generated.tsx" -exec sed -i '' "s|import \* as Operations from './|import * as Operations from \'../|" {} \;
find MUTATIONS_GENERATED_FOLDER -type f -name "*.generated.tsx" -exec sed -i '' "s|import \* as Operations from './|import * as Operations from \'../|" {} \;
`

this is an old bug, why it is not solved until now?

Hi everyone and thank you for the reports
Sorry but I'm not adding a lot here but just labeling it according to our new Contribution Guide and issue flow.
It seems like we are on stage 0.
Now in order to advance to stage 1 we need a reproduction in a repo or codesandbox.
Thanks for the great amount of details you provided on the issue, creating the reproduction shouldn't be too hard.
So the biggest benefit would be to advance to stage 2. In order to get there we'll need a failing test, would be great if someone could help progress the issues through the stages.
Thank you and sorry that this comment is not a complete solution (yet).


framework.dart's updateChildren isn't as flexible as it seems to be

If you use slots that aren't Elements it doesn't work as it passes the previousChild as the slot to updateChild.

The documentation does say that this is what it does. I'm going to close this Working As Intended. If you had a particular other resolution in mind instead let me know.


coverity issues

Found by neovim/neovim coverity:
4 new defect(s) introduced to neovim/neovim found with Coverity Scan.
1 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan.

New defect(s) Reported-by: Coverity Scan
Showing 4 of 4 defect(s)

** CID 143921:  Uninitialized variables  (UNINIT)
/tmp/luarocks_luarocks-source-mpack-1.0.2-0-140/libmpack-1.0.2/src/conv.c: 23 in mpack_pack_nil()

________________________________________________________________________________________________________
*** CID 143921:  Uninitialized variables  (UNINIT)
/tmp/luarocks_luarocks-source-mpack-1.0.2-0-140/libmpack-1.0.2/src/conv.c: 23 in mpack_pack_nil()
17       } while (0)
18
19     MPACK_API mpack_token_t mpack_pack_nil(void)
20     {
21       mpack_token_t rv;
22       rv.type = MPACK_TOKEN_NIL;
>>>     CID 143921:  Uninitialized variables  (UNINIT)
>>>     Using uninitialized value "rv". Field "rv.length" is uninitialized.
23       return rv;
24     }
25
26     MPACK_API mpack_token_t mpack_pack_boolean(unsigned v)
27     {
28       mpack_token_t rv;

** CID 143920:  Uninitialized variables  (UNINIT)
/tmp/luarocks_luarocks-source-mpack-1.0.2-0-140/libmpack-1.0.2/src/conv.c: 32 in mpack_pack_boolean()

________________________________________________________________________________________________________
*** CID 143920:  Uninitialized variables  (UNINIT)
/tmp/luarocks_luarocks-source-mpack-1.0.2-0-140/libmpack-1.0.2/src/conv.c: 32 in mpack_pack_boolean()
26     MPACK_API mpack_token_t mpack_pack_boolean(unsigned v)
27     {
28       mpack_token_t rv;
29       rv.type = MPACK_TOKEN_BOOLEAN;
30       rv.data.value.lo = v ? 1 : 0;
31       rv.data.value.hi = 0;
>>>     CID 143920:  Uninitialized variables  (UNINIT)
>>>     Using uninitialized value "rv". Field "rv.length" is uninitialized.
32       return rv;
33     }
34
35     MPACK_API mpack_token_t mpack_pack_uint(mpack_uintmax_t v)
36     {
37       mpack_token_t rv;

** CID 143919:  Null pointer dereferences  (FORWARD_NULL)
/tmp/luarocks_luarocks-source-mpack-1.0.2-0-140/libmpack-1.0.2/binding/lua/lmpack.c: 800 in lmpack_session_receive()

________________________________________________________________________________________________________
*** CID 143919:  Null pointer dereferences  (FORWARD_NULL)
/tmp/luarocks_luarocks-source-mpack-1.0.2-0-140/libmpack-1.0.2/binding/lua/lmpack.c: 800 in lmpack_session_receive()
794     {
795       int argc, done, rcount = 3;
796       lua_Number startpos;
797       size_t len;
798       const char *str, *str_init;
799       Session *session;
>>>     CID 143919:  Null pointer dereferences  (FORWARD_NULL)
>>>     Assigning: "unpacker" = "NULL".
800       Unpacker *unpacker = NULL;
801
802       if ((argc = lua_gettop(L)) > 3 || argc < 2)
803         return luaL_error(L, "expecting between 2 and 3 arguments");
804
805       session = lmpack_check_session(L, 1);

** CID 143918:  Incorrect expression  (ASSERT_SIDE_EFFECT)
/tmp/luarocks_luarocks-source-mpack-1.0.2-0-140/libmpack-1.0.2/binding/lua/lmpack.c: 182 in lmpack_objlen()

________________________________________________________________________________________________________
*** CID 143918:  Incorrect expression  (ASSERT_SIDE_EFFECT)
/tmp/luarocks_luarocks-source-mpack-1.0.2-0-140/libmpack-1.0.2/binding/lua/lmpack.c: 182 in lmpack_objlen()
176      * https://github.com/antirez/lua-cmsgpack/blob/master/lua_cmsgpack.c */
177     static mpack_uint32_t lmpack_objlen(lua_State *L, int *is_array)
178     {
179       size_t len, max;
180       int isarr, type, top;
181       lua_Number n;
>>>     CID 143918:  Incorrect expression  (ASSERT_SIDE_EFFECT)
>>>     Assignment "top = lua_gettop(L)" has a side effect.  This code will work differently in a non-debug build.
182       assert(top = lua_gettop(L));
183
184       if ((type = lua_type(L, -1)) != LUA_TTABLE) {
185         len = lua_objlen(L, -1);
186         goto end;
187       }

I will fix, thanks

@justinmk these are false positives:

The mpack_pack_{nil,boolean} functions don't initialize length, but the length is never read for these types.
The null pointer deference is not possible for properly initialized Sessions
The assert side effect doesn't matter since the value is only read in another assert.

I'd rather not initialize values and add extra checks just to make a tool happy, can we mark these false positives so they are ignored?

we can mark the false positives in coverity. Can an improperly-initialized session be signaled in some way other than a segfault?

clang 5.0 (not just coverity) also warns about the assert(top = lua_gettop(L)); . That code is also going to surprise anyone reading the code ...

Can you check if https://github.com/libmpack/libmpack-lua/pull/16 fixes the warning?


Can't access RPC

Hi I'm trying to set up a node using this repo, everything appears to be synced, and I'm having trouble accessing the RPC endpoint.
When I use the command http post http://<IP_ADDRESS>:8545 jsonrpc=2.0 id=1 method=eth_blockNumber params:='[]' I get this error:
http: error: ConnectionError: HTTPConnectionPool(host='<IP_ADDRESS>', port=8545): Max retries exceeded with url:
/ (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f47b443ce48>: Failed to establish a
new connection: [Errno 111] Connection refused')) while doing POST request to URL: http://<IP_ADDRESS>:8545/

The rpc node will listen on a docker IP address. If docker is correctly installed you can use http://mainnet_endpoint:8545
Alteratively find out the correct IP with:
docker inspect -f '{{range.NetworkSettings.Networks}}{{.IPAddress}}{{end}}' mainnet_endpoint


Fix webcaptions

Issue #, if available:
#673
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

skipping code review since this is a trivial change


YouTube Data API returns no channel on “Playlists.list” but on “Channels.list”

(This is a copy of this Stackoverflow question I've posted yesterday)
I'm not sure if this is an inconsistency of YouTube's Data API or just a bad naming of a YouTube error. I'm using PHP API client v2.8.3.
There's a public channel on YouTube without any content:
https://www.youtube.com/channel/UCvAHk_1_TMmq6SW0XA9BcgA.
If I query Channels:list (query) with this channel ID UCvAHk_1_TMmq6SW0XA9BcgA, I get the following response:
{
  "items": [
    {
      "id": "UCvAHk_1_TMmq6SW0XA9BcgA",
      "snippet": {
        "title": "level10store0"
      },
      "status": {
        "privacyStatus": "public",
        "isLinked": true,
        "longUploadsStatus": "longUploadsUnspecified"
      }
    }
  ]
}

If I query for all playlists of this channel via Playlists:list (query) I get this response:
{
  "kind": "youtube#playlistListResponse",
  "etag": "iziRvpOe3rkNWHlySTxLgAk4dJI",
  "pageInfo": {
    "totalResults": 0,
    "resultsPerPage": 5
  },
  "items": []
}

But using YouTube's PHP client with Playlist:list you'll get a response error with following information:
domain=youtube.playlist, reason=channelNotFound, location=channelId, locationType=parameter
Why does YouTube's PHP client give a channelNotFound error instead of an empty response like YouTube's data explorer does?
Update 1:
YouTube's data API delivered a channel on the OAuth handshake initialized by the user itself. My application asked for an offline access token for further requests. If I use this access token I experience the above mentioned behaviour. I think the token is valid as I use the identical process for many other users. If the token would not be valid I would have got an error message like global.authError or youtube.header.youtubeSignupRequired.
My code looks like this (which works for many other users):
$youtubeClient->playlists->listPlaylists(
    'snippet,contentDetails',
     array(
        'maxResults' => 50,
        'channelId' => $channelId,
        'pageToken' => $nextPlaylistPageToken,
        'fields' => 'etag,items(id,etag,snippet(publishedAt,channelId,thumbnails/default,title),contentDetails(itemCount)),nextPageToken'
    )
);

Hi @ninsky
First of all, I apologise for such a late response on the issue.
Perhaps your issue is long resolved by now.
But I did try the code snippet and also tried running the same on the Youtube API reference page. I was not able to replicate the issue with or without using the offline grant type.
What I mean is the PHP library and the api reference page, both returned the same result. I get an empty array when I try to use the following:
$service = new YouTube($client);
$playlists = $service->playlists->listPlaylists(
        'snippet,contentDetails',
         array(
            'maxResults' => 50,
            'channelId' => "UCvAHk_1_TMmq6SW0XA9BcgA",
            'fields' => 'etag,items(id,etag,snippet(publishedAt,channelId,thumbnails/default,title),contentDetails(itemCount)),nextPageToken'
        )
);

var_dump($playlists->getItems());

I don't see the channelNotFound error that you mentioned.
Please let me know if this was resolved for you or did you circumvent this at the time.
Thanks for the patience.

Hello @saranshdhingra
Thanks for your reply! My application performs thousands of requests per day. Currently I don't have the time to see whether this error still occurs. But I will come back to this issue in case I can reproduce it.
BR
ninsky

A similar issue is again seen. The YouTube API Reference page returns Channel Not Found for any public channel when we specify ChannelId.
And the python API behaves like mine=True when we use channelId= . There is a bug there too.

This issue is in reference to the PHP client library. You can file an issue for the python client here.
If you're seeing the issue on the APIs explorer, PHP client, AND python client, then it's unlikely the issue is in the client libraries (which is what we cover), and instead is an API Usage issue. I'd suggest asking your question on stack overflow instead.


undefined is not an object(evaluating 'item.props.selected')

always appears error message -------undefined is not an object(evaluating 'item.props.selected').
refers to TabNavigator.js:58

I have the same error,

Copying the example also gives me this error, any clue?

Did you solve. I've same issue

Maybe you just copy paste some code (like the native elements example).
So just remove the comment /* ... more tabs here */. Should works after that.

I got the same error!
and do you resolve it?

Will close since this issue is more than a year, feel free to a open a new if this is still a issue.


Update all.json

isecurewalletsdapp.com [<IP_ADDRESS>]
walletsdappconnection.com [<IP_ADDRESS>]
validateswallets.com [<IP_ADDRESS>]
walletsdapps.net [<IP_ADDRESS>]


Localise examples for user guide

This issue resolves
https://github.com/AY2122S2-CS2103T-T17-3/tp/issues/114.
Now, examples mimic how an insurance agent would use the application.

Codecov Report

Merging #115 (f66cb60) into master (9ed928c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #115   +/-   ##
=========================================
  Coverage     71.48%   71.48%
  Complexity      649      649
=========================================
  Files           110      110
  Lines          2104     2104
  Branches        216      216
=========================================
  Hits           1504     1504
  Misses          542      542
  Partials         58       58

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9ed928c...f66cb60. Read the comment docs.


Update: tag

Codecov Report

Merging #432 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #432   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         155    155
  Lines        1547   1547
=====================================
  Hits         1547   1547

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0fffce7...31da59b. Read the comment docs.


Documentation

We need more documentation on the main components of the simulator and a getting started kind of guide. Possibilities include:

[x] simple Doxygen html pages
[ ] wiki on ROS
[ ] wiki on guthub

Where can I find the documentation on "simple Doxygen html pages" ?


Concurrent flag evaluation bursts the back-end

Hello, consider the following program:
package main

import (
	"context"
	"fmt"
	"sync"

	gofeatureflag "github.com/open-feature/go-sdk-contrib/providers/go-feature-flag/pkg"
	"github.com/open-feature/go-sdk/openfeature"
)

var client *openfeature.Client

func main() {
	options := gofeatureflag.ProviderOptions{
		Endpoint: "http://localhost:1031",
	}

	provider, _ := gofeatureflag.NewProvider(options)

	openfeature.SetProvider(provider)

	// Create a new client
	client = openfeature.NewClient("app")

	wg := &sync.WaitGroup{}
	wg.Add(10)

	// Get the feature flag with multiple go routines
	for i := 0; i < 10; i++ {
		go deepInTheCode()
	}

	wg.Wait()
}

func deepInTheCode() {
	// Evaluate the feature flag
	v2Enabled, _ := client.BooleanValue(
		context.Background(), "v2_enabled", false, openfeature.NewEvaluationContext("foo", nil),
	)
	// Use the returned flag value
	if v2Enabled {
		fmt.Println("v2 is enabled")
	}
}

We know that goff provider caches the flag result.
However, during the first request, if there are concurrent jobs getting the flag value, we get the following effect:
You can see multiple concurrent unnecessary requests coming from the application. Think when we have like thousands of go routine workers.

I propose here to leverage https://pkg.go.dev/golang.org/x/sync/singleflight
You may argue that if the context is the same, the flag should be retrieved in advance, but think of applications that would need a big refactoring to allow that to happen, or even parts of the application that are not too close that may need the same flag in the same moment.
Using a single flight group would be an elegant way to prevent that from happening coming from any part of the application, by using an unique key based on the evaluation context, just like the cache does.
The question is in which layer we should apply this logic:

Per provider approach
In that case, the provider is responsible to take care of the cache warm up and it will need to prevent bursts like that. In our example, we might add the code to the go-feature-flag provider, using the same cache key strategy for the flight key.

ofprep provider approach
The single flight would always occur in the ofprep provider.

Client approach (Provider independent) approach
We could add the single flight directly to the client thus avoiding bursts on any kind of back-end provider.

For any of the approaches we might want to move the cache key strategy to a func in FlattenedContext struct and use that both for caching and for single flight key.

If I have understand correctly, you are talking about concurrent evaluation happening at the same time with the same context and the same flag.
And what you want to avoid is to do all the calls to remote system (or even local evaluation for other type of providers) in parallel but rather use singleflight to do it once for everyone?
Based on all your options I am not sure where it makes the most senses to do it.
Since GO Feature Flag provider is just a wrapper of the OFREP Provider, I am curious to have the point of view of @Kavindu-Dodan, @lukas-reining or @toddbaert about that.
I am also wondering if it should be part of the SDK or this is up to the provider to decide?

Could we perhaps build a "wrapper" provider that does this, and takes another provider in it's constructor? That way we would have some composibility? A SingleFlightProvider which then calls whatever "real" provider it wraps?

@toddbaert This is definitely something I can do in the GO Feature Flag wrapper, but for OFREP if this is something we see often, could it be a configuration of the provider directly maybe?


rename roles.csv to members.csv

It is a list of membership, which just happens to show the role for each user. This name will be easier for people to understand.

Is there a distinction between "member" and "user"? If not, should probably rename DATA_DIR/users/ to members too.

No, you are correct. In this case, users are assumed to be members. A gitguild user might investigate a repository that they are not a member of, for instance to do auditing checks, or consume publicly available information. But that distinction is quite fine, and not really relevant for the code base atm.

I realized the "user" terminology is embedded throughout the code, and really from the programmer's view I prefer it to "member". And who knows, maybe some time we'll want to reference users that aren't strictly members. So, sticking with the original request to just rename the file which is explicitly a membership list.

Need to update README for the roles.csv rename.

@welshjf I'm closing this as the main merge was done. Please do modify the README, though!


add all event attribute types

add all event attribute types
add predicate about class and thread

Squashed and applied. Thanks a lot, @HashZhang!


"TODO" in gemspec template breaks methadone on generation

Hi all!
I've been working on a methadone issue, where methadone is using bundle gem to generate a gemspec here but it breaks for two reasons:
1.Bundle isn't happy about TODO in the gemspec, it needs to be todo or some other nonecaps thing.
2. The spec.homepage need to either be empty or some site-like pattern like https://rubygems.org
I think it's from lines 12-14 in the newgem template that bundle gem uses.
There result is you need to go in and make some replacements before you can get methadone to work.
Is this intentional? Looking forward to hearing from you 😄
also great job @indirect in your ruby conf video! it was a fun and informative watch 👍

@mbigras rubygems does gemspec validation, which includes a regex for TODO or FIXME (src). and spec.homepage does a similar thing a few lines down. So yes, its intentional.
Looking over the methadone issue though, it looks like ideally you want to use a bundle gem flag that creates a new gem that's "ready to go" ? (thinking about this is why I preemptively added the feature request label, heh)

bundle gem purposely leaves the TODOs in, as well as things to complete in the README, etc. if you're looking for a more customizable way to automate create gems, there are several options , including something called ore, I believe.

Hi @lynnco @segiddins!
Thank you for getting back to me. Okay I just wanted to check, it makes sense for the TODOs to be the default behavior and to check the homepage for a valid url.
A --no-todos switch and a --default-website switch (that would read the a site from ~/.bundle/config) could be cool, but super low priority :P maybe hense the triage flag.
I made a small script for anyone interested:
public_site = "https:\/\/github.com/mbigras"
gemspec = Dir['*.gemspec'].first

gemspec_contents = File.open(gemspec).read
transformed_gemspec_contents = gemspec_contents
.gsub(/TODO: /, '')
.gsub(/^.*\('rake'\)\n/, '')
.gsub(/Put your.*here\./, public_site)

File.write(gemspec, transformed_gemspec_contents)


build for target=arm...musl fails

I tried to x-compile on a x86_64 linux system
cargo build --target=arm-unknown-linux-musleabihf
cargo build --target=arm-unknown-linux-musleabi
cargo build --target=armv7-unknown-linux-musleabihf

all are failing with error "Compiler error reporting is too harsh for ./configure (perhaps remove -Werror)", e.g.
   Compiling libz-sys v1.0.18 (https://github.com/alexcrichton/libz-sys.git#8eb6f9e8)
error: failed to run custom build command for `libz-sys v1.0.18 (https://github.com/alexcrichton/libz-sys.git#8eb6f9e8)`
process didn't exit successfully: `/home/rainer/test/rust/lg2/lg2_test/target/debug/build/libz-sys-79688570e7235f69/build-script-build` (exit code: 1)
--- stdout
cargo:rerun-if-env-changed=LIBZ_SYS_STATIC
cargo:rerun-if-changed=build.rs
OPT_LEVEL = Some("0")
TARGET = Some("arm-unknown-linux-musleabihf")
HOST = Some("x86_64-unknown-linux-gnu")
TARGET = Some("arm-unknown-linux-musleabihf")
TARGET = Some("arm-unknown-linux-musleabihf")
HOST = Some("x86_64-unknown-linux-gnu")
CC_arm-unknown-linux-musleabihf = None
CC_arm_unknown_linux_musleabihf = None
TARGET_CC = None
CC = None
HOST = Some("x86_64-unknown-linux-gnu")
CROSS_COMPILE = None
TARGET = Some("arm-unknown-linux-musleabihf")
HOST = Some("x86_64-unknown-linux-gnu")
CFLAGS_arm-unknown-linux-musleabihf = None
CFLAGS_arm_unknown_linux_musleabihf = None
TARGET_CFLAGS = None
CFLAGS = None
DEBUG = Some("true")
running: "./configure" "--prefix=/home/rainer/test/rust/lg2/lg2_test/target/arm-unknown-linux-musleabihf/debug/build/libz-sys-a4825891e6086beb/out"
Compiler error reporting is too harsh for ./configure (perhaps remove -Werror).
** ./configure aborting.

command did not execute successfully, got: exit code: 1

Some additonal setup or parameters are needed for cross-compiling ?

configure.log reports
./configure: 182: ./configure: arm-linux-musleabihf-gcc: not found

Now I tried compiling with
1) CC=arm-linux-gnueabihf-gcc cargo build --target=arm-unknown-linux-musleabihf
2) CC=arm-linux-gnueabihf-cpp cargo build --target=arm-unknown-linux-musleabihf
3) CC=arm-linux-gnueabihf-g++ cargo build --target=arm-unknown-linux-musleabihf

does succeed.
reports in configure.log

arm-linux-gnueabi-cpp: fatal error: ‘-c’ is not a valid option to the preprocessor

outputs a bunch of compile errors on console

Is 1) a proper solution to compile the crate  ? (I thought 'cargo build target=...' is sufficient )
Or do I need to get "arm-linux-musleabihf-gcc" ? If yes where can I get it ? (I'm on Ubuntu 17.10)

Ok. My expectation was same as expresssed here https://github.com/rust-lang/cargo/issues/4741#issuecomment-349750036
From discussion there I can see, currently it is still necessary to pass via environment variables proper information about compiler, include path, library path ... beside cargo's 'target=...' option
So either "CC=arm-linux-gnueabihf-gcc" or "CROSS_COMPILE="arm-linux-gnueabihf-" are good to pick proper cross-compiler.
I assume "CROSS_COMPILE=..." should be preferred ?
And in case wrong header files are used I can fix it by using "CFLAGS=I..." (faced that issue here https://github.com/tailhook/vagga/pull/482#issuecomment-354473114)
Before closing this thread it would be nice if someone can confirm if my understanding is correct now or point out where I'm still wrong.
Thanks.


[Question] [support needed] Error Response Body is empty

I am trying to decode error response body but got always null:
code specific ErrorCodes  error Exception classes are generated properly but with empty body:

Feign's OkHttp client is used.
Where is my bad?
Used versions:

Feign: 10.2.3
annotatiion-error-decoder: 1.2.0

P.S. Another developers from UA reported feign-error-decoder does not work properly.
See also my question in chat.

@hellboy81 can you try:
There's one part in the exception generator that I'm not terribly proud of:
  private Object resolveBody(Response response) {
    if (bodyType instanceof Class<?> && ((Class<?>) bodyType).isInstance(response)) {
      return response;
    }
    try {
      return bodyDecoder.decode(response, bodyType);
    } catch (IOException e) {
      // How do we log this?
      return null;
    } catch (DecodeException e) {
      // How do we log this?
      return null;
    }
  }

you see that it basically tries its best to decode the body, but if it fails, it actually does so silently (yuck).
That said - Feign itself doesn't decide on a logger for you... which begs the question: how to best log?
I expect that ErrorResponse is likely being returned in a format you don't expect or that your POJO doesn't fit to. You can perhaps debug by having the constructor just take feign.Response and capture the body/decode it yourself just to see what is happening.
The other option is a little more intrusive (Support some form of logging - while still not trying to be opinionated - much like what Feign itself tries to do). If you have a proposal (maybe we include a Logger class/object that logs the error, and if available, the body or so) then I'll be happy to implement it (or even, if you are up for it, have you implement it and I'll review).
Thoughts?


Fix: use sorted cidr list

Codecov Report

Merging #1156 (3f61198) into master (e2d526c) will decrease coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1156      +/-   ##
==========================================
- Coverage   44.74%   44.62%   -0.12%
==========================================
  Files         485      476       -9
  Lines       29494    29395      -99
==========================================
- Hits        13197    13118      -79
+ Misses      14890    14869      -21
- Partials     1407     1408       +1

Impacted Files
Coverage Δ

app/router/condition_geoip.go
90.17% <100.00%> (ø)

testing/servers/tcp/tcp.go
81.81% <0.00%> (-5.46%)
:arrow_down:

transport/internet/udp/hub.go
73.61% <0.00%> (-4.17%)
:arrow_down:

transport/internet/system_listener.go
50.00% <0.00%> (-4.00%)
:arrow_down:

transport/internet/kcp/dialer.go
80.39% <0.00%> (-3.93%)
:arrow_down:

testing/scenarios/common.go
70.68% <0.00%> (-3.45%)
:arrow_down:

transport/internet/system_dialer.go
46.83% <0.00%> (-2.54%)
:arrow_down:

transport/internet/kcp/connection.go
69.61% <0.00%> (-2.21%)
:arrow_down:

transport/internet/kcp/listener.go
81.18% <0.00%> (-1.99%)
:arrow_down:

transport/internet/kcp/receiving.go
98.65% <0.00%> (-1.35%)
:arrow_down:

... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2d526c...3f61198. Read the comment docs.


More robust merges

Currently, there are two situations that Irmin do not handler very well:

trying to merge when no parents in common. This usually do not happen, but why not try to support it (even if it is not very efficient). Also, it might be necessary to handle if we deal with partial history (see #21). This means adding a merge2 function in user-defined contents.

trying to merge where multiple common ancestor exists. Currently, Irmin fails (see https://github.com/mirage/irmin/blob/master/lib/core/irminCommit.ml#L180) but we should be able to 3-merge all the pair of parents recursively to get a unique ancestor.

Fixed in 0.9.0


Add checkout and inline npm version to fix HybridApp deploys

Details

We were not passing a build version to HybridApp because we did not have any code to grab a version from, this adds the checkout and additionally cleans up the code a bit.

Fixed Issues

https://github.com/Expensify/App/actions/runs/9603932566/job/26488267595
Tests

CP this to staging and verify the HybridApp version is <IP_ADDRESS>

QA Steps

N/A

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
N/A - No source code changes

Android: mWeb Chrome
N/A - No source code changes

iOS: Native
N/A - No source code changes

iOS: mWeb Safari
N/A - No source code changes

MacOS: Chrome / Safari
N/A - No source code changes

MacOS: Desktop
N/A - No source code changes

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 9.0.0-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 9.0.0-9 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


added macOS to default certificate authority bundle chart

@Jiasli, I don't have a way to confirm this macOS path.  Can you please review?  Thank you.
This PR is in response to issue #3007

Docs Build status updates of commit a613a56:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/use-cli-effectively.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Nice change! @dbradish-microsoft, may I know how you found that location?

@jiasli, GitHub "issue" suggestion.


Chore: buildExamples command, CI, http-server example tests

Follows https://github.com/corazawaf/coraza/pull/420, closes https://github.com/corazawaf/coraza/issues/419.
In this PR:

buildExamples is added to the magefile. It iterates over all the (potential) examples under /examples/ and builds them
buildExamples is executed inside the CI to prevent commits that break any example
A couple of tests are added for the HTTP-server example.

Thank you for contributing to Coraza WAF, your effort is greatly appreciated
Before submitting check if what you want to add to coraza list meets quality standards before sending pull request. Thanks!

Make sure that you've checked the boxes below before you submit PR:

[ ] My code includes positive and negative tests.
[ ] I have an appropriate description with correct grammar.
[ ] I have read Contribution guidelines, maintainers note and Quality standard.
[ ] My code is properly linted and passes pre-commit tests.

Thanks for your contribution :heart:

Codecov Report
Base: 77.00% // Head: 77.00% // No change to project coverage :thumbsup:

Coverage data is based on head (733c03d) compared to base (53ed3d0).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           v3/dev     #422   +/-   ##
=======================================
  Coverage   77.00%   77.00%
=======================================
  Files         136      136
  Lines        5975     5975
=======================================
  Hits         4601     4601
  Misses       1106     1106
  Partials      268      268

Flag
Coverage Δ

no-tinygo
77.12% <ø> (ø)

overall
77.00% <ø> (ø)

tinygo
75.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Thank you for the review @jcchavezs! Marking as ready for broader approval


expression-file doesn't seem to work; 'Couldn't load module "undefined"'

I'm trying
dts-gen --expression-file ./index.js

But I get

Couldn't load module "undefined". Please install it globally (npm install -g undefined) and try again.

Which doesn't make much sense.
How can I generate a .d.ts file for my library without having to globally install it first?

Hi, any update on this?
I'm also facing similar issue while trying to generate .d.ts files

I'm in the same boat. Any suggestions on what might be going wrong? (If it's caused by a specific line in the .js file, a line number would be great...)

and your solution guys was ?

@DavidNorena  It's likely that there are relative path require() statements in expression-file, and would not be correctly resolved.
Just transform to absolute path to do the tricks, for example:
$ dts-gen -e "require('./dist/umd/emoutils.js')"
Couldn't load module "undefined". Please install it globally (npm install -g undefined) and try again.
# Change to Absolute Path
$ dts-gen -e "require('/absolute-path-to/emoutils.js')"
Wrote 52 lines to dts_gen_expr.d.ts.

Or dive into source and specify a correct cwd to fix this issue.


Copy text

Would be nice to have a way to copy text from posts and comments.
Selecting and copying text might be impossible currently due to Flutter ( https://github.com/flutter/flutter/issues/5422 ) but we can build something that when holding  top, shows a copy text dialog for now.

Can't remember which app, but some just copy the text automatically if you long-tap it (i.e. it doesn't show a "copy text" button when long tapping, but just copies immediately). Is a pop-up button or instant copy preferrable?
Would it be a good idea to add "Copy text" to the post's "three-dot menu" as well, or is that overkill?

Oh instant copy sounds better! We can show a toast to show that the text was copied :)

Fixed with #214 !


Classic logging is broken

I haven't looked at the code in depth, but I believe https://github.com/urfave/negroni/commit/2cb24dac5043933255b7079ac36533e331c27b64 introduced a bug that made classic logging stopped working
I remember that the negroni printed the status code, the path, the response time and the timestamp of each request with that logger and now it only prints an incorrectly timestamp.
This is what I get from running the example in the README
marcos@marcos-xps:~/Projects/gopath/src/github.com/marcosnils/gotest$ go run main.go
[negroni] 2006-01-02T15:04:05Z07:00
[negroni] 2006-01-02T15:04:05Z07:00
[negroni] 2006-01-02T15:04:05Z07:00
[negroni] 2006-01-02T15:04:05Z07:00
[negroni] 2006-01-02T15:04:05Z07:00
[negroni] 2006-01-02T15:04:05Z07:00
[negroni] 2006-01-02T15:04:05Z07:00

Should be fixed by #164; apologies that that one slipped through. I'm in the process of adding a test to catch it in the future.


KAFKA-16375: Fix for rejoin while reconciling

This PR includes a fix to properly identify a reconciliation that should be interrupted and not applied because the member has rejoined. It does so simply based on a flag (not epochs, server or local). If the member has rejoined while reconciling, the reconciliation will be interrupted.
This also ensures that the check to abort the reconciliation is performed on all the 3 stages of the reconciliation that could be delayed: commit, onPartitionsRevoked, onPartitionsAssigned.

Hey @lucasbru, could you please take a look at this one when you have a chance? Thanks!

Thanks both for the feedback. I pushed the changes addressing all the suggestions.


Initial attempt at supporting different matchers

Signed-off-by: David McKay<EMAIL_ADDRESS>I don't believe this is optimal, I'm really looking for some guidance.
I believe do_match should be defp, but I don't know if that is possible because assert_delivered_with is a macro

I just realized it would probably be helpful to update the docs so people know about this. I'll add an issue so this can be done in a separate PR. Thanks for the awesome work @rawkode


Segemented Messages / Handling

Hi Team,
In the documentation it does not mention limitations of the MQ Connector.
We are receiving messages which are segmented. Logic app connector is not able to receive those messages.
App runs for 15/20 mins and then returns a blank message.
Do you think segmented messages are supported?
Regards,
Vishal

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 42e7fab4-1469-22df-2618-b6b82b09df6d
Version Independent ID: 4d8ae394-5c92-7949-db5f-99884de55c21
Content: Connect to IBM MQ server - Azure Logic Apps
Content Source: articles/connectors/connectors-create-api-mq.md
Service: logic-apps
GitHub Login: @ChristopherHouser
Microsoft Alias: chrishou

Hello @vishalaj1
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

We do not currently support segmented messages.   Are you trying to send message larger than 100 MB?

Messages are not more than 100MB however they are segmented messages.

Typically, segmentation is to work around MQ's 100 MB message limit.  Any message under that size can be sent as a whole message.  I'll added this to our customer asks for consideration in future improvements.

Thanks, I will ask the supplier of MQ if they can remove segmentation. It is not in same network and we are connecting logic app via azure vnet to on prem vpn tunnel to supplier MQ. Probably that can be a limitation on transmission from supplier side.
If it is segmented right now the messages are deleted when we use receive messagein logic app. eventhough the receive message looks like in hang stage it finishes after 15-20 mins with blank message and message is deleted from queue.

Also note we are only doing new features in the built-in MQ Connector (Logic Apps Standard).   If the supplier cannot make the changes let me know.

Yes we are using built-in connector in logic app standard.
Sure I have meeting with them on Monday.
Will send feedback on Monday.
Thanks for looking into this.

@ChristopherHouser
Now we checked with supplier. They are not sending segmented messages, however they still have segmentation flag.
In this case they expect that we should not get any segmented messages if size is below 100MB.
The current size of messages sent to us is around 30-40MB. Logic app 'Receive Multiple Messages' runs for  20-30 mins but we receive blank message as output from this action.
I am not sure it should take this much time in general for 30-40 MB files.

@vishalaj1
I think the best route would be to open a support case, so we diagnose the cause of receive failure.

Hi,
Thanks for your feedback and help. Now we disabled the setting from the IBM MQ on supplier side.
We are receiving messages from the source now. So issue is resolved for now.
Is there a way that we are informed when the segmentation feature is available for standard connector.
Regards,
Vishal

Wanted to check whether doc updates are still necessary? If not, we can close this thread as this forum is dedicated to doc issues. Thanks!

@ecfan it would be good to mentorn that segmentet Messages ar emot supporten as of now in documentation

@vishalaj1: Thanks, a doc update should be live later today.
@ChaitanyaNaykodi-MSFT: #please-close

#assign:@ecfan


As described in issue, replaced separate Sprintf/Printf with single P…

…rintf call.  Other than occurrences in usage.go, I couldn't find any other usages do anything similar, so makes this a pretty simple change.  Everything compiles and runs just fine.
Signed-off-by: Scott C Sosna<EMAIL_ADDRESS>

Which issue are you making this change against?

EF-1659
https://github.com/edgexfoundry/edgex-go/issues/1659
On 8/20/19 16:31, Brandon Forster wrote:

Which issue are you making this change against?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/edgexfoundry/edgex-go/pull/1693?email_source=notifications&email_token=AG2LPL5QTPE6WC26Y7IZ6OTQFRPBXA5CNFSM4IN5BUM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4XXDPI#issuecomment-523203005,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG2LPL5773TFWIAMDUIDIA3QFRPBXANCNFSM4IN5BUMQ.

Thanks! Rebase to master and I'll approve this.

Codecov Report

Merging #1693 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1693   +/-   ##
=======================================
  Coverage   34.67%   34.67%
=======================================
  Files         139      139
  Lines       11770    11770
=======================================
  Hits         4081     4081
  Misses       7488     7488
  Partials      201      201

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 38d7c2a...7588b85. Read the comment docs.

Will when I get home tonight
Sent from my iPhone

On Aug 20, 2019, at 4:50 PM, Brandon Forster<EMAIL_ADDRESS>wrote:
Thanks! Rebase to master and I'll approve this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Rebased and pushed.
-scs
On 8/20/19 16:50, Brandon Forster wrote:

Thanks! Rebase to master and I'll approve this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/edgexfoundry/edgex-go/pull/1693?email_source=notifications&email_token=AG2LPL3D4J7M3PPRBLTP5MLQFRRKHA5CNFSM4IN5BUM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4XYSYQ#issuecomment-523209058,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG2LPL6S4DR4NO7CCF732UTQFRRKHANCNFSM4IN5BUMQ.

rescan


Allow --custom when adding transformers

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/3407
Originally created by @aaronsteers on 2022-04-18 02:38:42

I discovered in my dogfooding project (https://gitlab.com/meltano/meta/-/issues/234), that adding --custom transformers is explicitly disallowed as of now and this is problematic for the path we are going with #3298.
We should allow transformer plugins to be defined via --custom as we do today for extractor plugins and other types.

View 8 previous comments from the original issue on GitLab


0.6.2 call another .feature file not work

When I try data-driven testing by looping over a .feature file for each object in a JSON array, it shows
" Scenarios (* undefined)

Steps (* undefined)
0m0.000s".
Then I try call read('*.feature') without dynamic parameters(very simple, see the attached screenshots), and it doesn't work neither(shows the same message as above). What's wrong with that?

Console Output
Feature: Call another feature file
Scenario: calling a feature file                     # E:/eclipse-workspace/karatedemo/src/test/java/CallFeatureFile.feature:3
* def result = call read('SubFeatureFile.feature')
* print result.response
1 Scenarios (1 undefined)
2 Steps (2 undefined)
0m0.000s
You can implement missing steps with the snippets below:
@Given("^def result = call read\('SubFeatureFile\.feature'\)$")
public void def_result_call_read_SubFeatureFile_feature() throws Throwable {
// Write code here that turns the phrase above into concrete actions
throw new PendingException();
}
@Given("^print result\.response$")
public void print_result_response() throws Throwable {
// Write code here that turns the phrase above into concrete actions
throw new PendingException();
}

I think you have the wrong eclipse plugin: please refer to the doc: https://github.com/intuit/karate#troubleshooting-cucumber-ide-support

Thanks  Peter. Yes, I forgot to change the Run Configure(change the Glue to com.intuit.karate) for the  newly added feature file. I fixed that problem. But calling another .feature file still doesn't work. It shows "demo.feature". However, the called .feature file could run standalone successfully. Is there anything mis-used?

Thanks Peter. Yes, I forgot to change the Run Configuration 'Glue' for the newly added feature files.


Fix(footer): Adjusted to always get current year

I was scrolling thought the lang page and I saw the year was not in the current year.
Added a function that places the current year as copyright.
Don't know hot it works in the US...
In pages from my country I've added this function to always place the current year in this section of the footer.
Also It could be a small adjust to place year of launch -
If there is anything that needs to be changed I'm here to help :)

fixed with #521

Thanks for taking this on though. This PR had a diff in the package-lock.json, so I merged the equivalent PR mentioned above.


[TASK-7] As a user, I can view the forget password screen

Task Card - https://github.com/rafayet-monon/surveyor/projects/1#card-43745447
What happened
Creating the forgot password screen with alert component styles.
Insight

[x] Write Forgot Password Component CSS
[x] Create Forgot Password Screens

Proof Of Work

Expected Design from Figma

Implemented Design

Responsiveness

Test Cases

This CI should pass

Fails

:no_entry_sign:

Danger failed to run dangerfile.js.

Error ReferenceError
fs is not defined
ReferenceError: fs is not defined
    at Object.<anonymous> (dangerfile.js:10:24)
    at Module._compile (internal/modules/cjs/loader.js:1137:30)
    at Object.requireFromString [as default] (/home/runner/work/surveyor/surveyor/node_modules/require-from-string/index.js:28:4)
    at Object.<anonymous> (/home/runner/work/surveyor/surveyor/node_modules/danger/distribution/runner/runners/inline.js:144:63)
    at step (/home/runner/work/surveyor/surveyor/node_modules/danger/distribution/runner/runners/inline.js:32:23)
    at Object.next (/home/runner/work/surveyor/surveyor/node_modules/danger/distribution/runner/runners/inline.js:13:53)
    at /home/runner/work/surveyor/surveyor/node_modules/danger/distribution/runner/runners/inline.js:7:71
    at new Promise (<anonymous>)
    at __awaiter (/home/runner/work/surveyor/surveyor/node_modules/danger/distribution/runner/runners/inline.js:3:12)
    at exports.runDangerfileEnvironment (/home/runner/work/surveyor/surveyor/node_modules/danger/distribution/runner/runners/inline.js:105:136)

Dangerfile
 5| const stylelint = require('danger-plugin-stylelint')
 6|
 7| // ESLINT
 8| const esLintFile = 'eslint-output.json';
 9| const esLinterOutput = fs.readFileSync(esLintFile).toString();
--------------------------^
10| const esLintJson = JSON.parse(esLinterOutput);
11|
12| if (Object.keys(esLintJson).length !== 0) {
13|   for (let i = 0; i < esLintJson.length; i++) {

  Generated by :no_entry_sign: dangerJS against fad138514332bc0a89033f2c003de3aeb81107ce


Alexey-Ben meeting 2018-09-05

Two week horizon:

@ashiklom to request access to PIC
@bpbond to request project PIC allocation
Talk with VCU folks about standing up a Slack
Ben to forward Alyssa's GCAM meeting invite email, and Alexey will attend (mid-October)
Set up standing meeting time

Two months:

Hector and Hector-related papers, including Hartin et al 2015 and 2016, Lynch et al. 2017, Kravitz et al. 2018, and Link et al. 2018
Clone and build: https://github.com/JGCRI/hector
Touch base with Robert (current reset/MCMC work) and Mike (vis-à-vis PEcAn) re plans
Probably 10% of time?
Alexey PhD chapters–keep 'em moving. Ch. 2 (rework for resubmission); Ch. 3 (add data, consider venue); Ch. 4 (ED radiative transfer; AGU talk). Another 10% of time (and obviously longer than two months)
Work towards setting up a PEcAn node on PIC. This will involve some research and talk with IT support (especially wrt always-running daemon) (40%)
Begin assembling any necessary data for first-year FoRTE paper (10%)

Two years:

Pauling Fellowship, probably fall 2019, but @ashiklom to investigate whether makes sense to apply this fall
Think about summer 2019, and talk with @atkinsjeff re fieldwork plans and possible cross-lab collaborative work (10%)
First-year FoRTE paper running PEcAn at UMBS
Mentoring, as needed, for junior staff

Other:

Discussed Ben's time allocation–very low on FoRTE
Charging protocols
Alexey to think about 2019 meeting(s) to attend
VCU Google Drive

Other stuff?

Week 1 updates:

Hector

Read Hartin et al. 2015 and 2016 papers
Downloaded, compiled, and successfully ran tests
Generated some output and plotted with R
Opened two small PRs -- one documentation fix, and one Makefile revision. Also, one wiki addition related to outputs bug.

FoRTE -- No updates
Dissertation residuals

Selected Ecology Letters for Chapter 1 resubmit. Reviewed and took notes on requirements.

Other

PNNL onboarding and training
In contact with PNNL IT/Research Computing about a cluster node for running PEcAn
Set up instance of PEcAn on my work machine for local testing and development while PIC discussions are ongoing


NA OFI: hostname not working with starting an engine

My objective is to make the server run on a specific port. Seems like explicitly specifying the address to use should work. However, it does not work is I use hostname instead of IP address or interface name when specifying the address for the server.
#include <unistd.h>
#include <thallium.hpp>
#include <string>

namespace tl = thallium;

/**
 * This is the ParaT server executable.
 */
int main(int argc, char* argv[])
{
  char buffer[256];
  gethostname(buffer, 256);

  std::ostringstream str;
  str << "tcp://" << buffer << ":11111";
  tl::engine myEngine(str.str(), THALLIUM_SERVER_MODE);
  std::cout << "requested address: " << str.str() << std::endl;
  std::cout << "server running at address: " << myEngine.self() << std::endl;
  return 0;
}

The output from this is as follows:
requested address: tcp://miron:11111
server running at address: ofi+tcp;ofi_rxm://<IP_ADDRESS>:33499

Platform (please complete the following information):

System description:   Ubuntu 20.04.3 LTS
Compiler version: GCC 9.3.0
Plugin and protocol used [e.g. ofi, psm2]
Dependency version: libfabric-1.13.0

@utkarshayachit PR #537 should fix your issue, I just merged it to master. Would you be able to try it out?

thanks for the fix...it still fails for me; although now I get an error message rather that it just picking a random port.
Here's the output from the same test code from the issue.
# [31877.645652] mercury->fatal: [error] /tmp/utkarsh/spack-stage/spack-stage-mercury-master-tubj37zytqb2mcdmpbrhiqe4mvbexaro/spack-src/src/na/na_ofi.c:2034
 # na_ofi_domain_open(): No provider found for "tcp;ofi_rxm" provider on domain "miron"
[error] Could not initialize hg_class
terminate called after throwing an instance of 'thallium::margo_exception'
  what():  [/home/utkarsh/Kitware/Mochi/spack/opt/spack/linux-ubuntu20.04-broadwell/gcc-9.3.0/mochi-thallium-develop-yufqz3w5lpooqe2rmafnxkynt7mr4kyc/include/thallium/engine.hpp:180][margo_init_ext] Could not initialize Margo

Thanks. Alright I think we'll get to the bottom of it though. Can you please set HG_LOG_LEVEL=warning HG_LOG_SUBSYS=na and rerun your test, that will give us more details. Also the output of fi_info on your system would be helpful. I am not entirely sure yet until I see the logs but it looks like your hostname cannot be resolved for some reason.

Also thinking more about it, you might want to double check also that your /etc/hosts does not associate your hostname to an interface that is down or something like that. I can't really think of anything else that would be significantly different on your system compared to the ones we use.

here are the results
# [481.582259] mercury->na: [error] /tmp/utkarsh/spack-stage/spack-stage-mercury-master-tubj37zytqb2mcdmpbrhiqe4mvbexaro/spack-src/src/na/na_ip.c:212
 # na_ip_check_interface(): No ifa_name match found for IP
# [481.582289] mercury->cls: [warning] /tmp/utkarsh/spack-stage/spack-stage-mercury-master-tubj37zytqb2mcdmpbrhiqe4mvbexaro/spack-src/src/na/na_ofi.c:3762
 # na_ofi_initialize(): Could not find matching interface for miron, attempting to use it as domain name
# [481.583238] mercury->fatal: [error] /tmp/utkarsh/spack-stage/spack-stage-mercury-master-tubj37zytqb2mcdmpbrhiqe4mvbexaro/spack-src/src/na/na_ofi.c:2034
 # na_ofi_domain_open(): No provider found for "tcp;ofi_rxm" provider on domain "miron"
# [481.583258] mercury->cls: [error] /tmp/utkarsh/spack-stage/spack-stage-mercury-master-tubj37zytqb2mcdmpbrhiqe4mvbexaro/spack-src/src/na/na_ofi.c:3834
 # na_ofi_initialize(): Could not open domain for tcp;ofi_rxm, miron
# [481.583270] mercury->cls: [error] /tmp/utkarsh/spack-stage/spack-stage-mercury-master-tubj37zytqb2mcdmpbrhiqe4mvbexaro/spack-src/src/na/na.c:339
 # NA_Initialize_opt(): Could not initialize plugin
[error] Could not initialize hg_class
terminate called after throwing an instance of 'thallium::margo_exception'
  what():  [/home/utkarsh/Kitware/Mochi/spack/opt/spack/linux-ubuntu20.04-broadwell/gcc-9.3.0/mochi-thallium-develop-yufqz3w5lpooqe2rmafnxkynt7mr4kyc/include/thallium/engine.hpp:180][margo_init_ext] Could not initialize Margo
fish: “env HG_LOG_LEVEL=warning HG_LOG…” terminated by signal SIGABRT (Abort)

cat /etc/hosts
<IP_ADDRESS> view-localhost
<IP_ADDRESS>       localhost
<IP_ADDRESS>       miron

Thanks, I believe the third line in your /etc/hosts file is causing the issues you're having. You should either remove it or have your permanent IP address assigned instead as documented there: https://www.debian.org/doc/manuals/debian-reference/ch05.en.html#_the_hostname_resolution

Having said that, we should probably also be able to support this type of loopback IP, I'll have a look to see if we can also do that. In that case I'd expect you to have miron:11111 resolved as <IP_ADDRESS>:1111, probably not the IP you'd want anyway but we should somehow support it.

FWIW, removing it does indeed seem to solve the issue

Great, thanks for confirming.


Can you provide a full example code with Payment Elements with create a payment intent first (client secret ..)

thanks for the full description..

Also we need a full payment element guide for this.. like card example...

Also we need a full payment element guide for this.. like card example...thnks

await elms.value.instance.confirmCardPayment(clientSecret, { 							payment_method: { 								card: cardNumber 							} 						})

https://stripe.com/docs/payments/accept-a-payment?ui=elements
I'm gonna provide a full example, which requires stripe account and payment intent (server-side), but really tight on time right now
Basically, you need client_secret, it then passed as property of options to StripeElements. Then you'll be able to call confirmPayment on stripe instance.

@softbeehive If you wanna assign this to me I can probably submit a PR with an example for using Payment Elements this weekend.

@yashBhosale you are welcome to submit a PR at any time
Here is some guidance:

Create PaymentElement.vue in examples folder
Use Vue Composition API (preferably)
Test payment type element
Add Payment Type section to README.md before "Types"

does anyone have a working example with payment intent??

Folks this is a cool package but also really confusing. Can anyone help provide a full example of how one can submit a payment to stripe and redirect to another page afterwards? Thanks so much. Best, Newb

Payment intent requires server-side code that depends on the language of your choice. I can totally see the value of having comprehensive guide on how to do that. And I can see why people struggle with its complexity.
But I realize this is outside of scope of this package, which has a single goal: integrate stripe.js with vue component lifecycle. The reason I didn't make type="payment" the default is that it requires extra work. I suggest asking Stripe for the vue guide, they have resources and capacity to offer it.
Community projects like this receive little to no support and I'm not first to say this but this is an uncomfortable truth about open-source. If someone is willing to invest time + energy and make the guide centered around vue integration of the payment element, I'm happy to link it in readme. I hope it explains the situation.

Quick guide
To start with payment element

Grab clientSecret from the payment intent
Pass it to elements-options

<template>
<StripeElements
  ...
  :elements-options="elementsOptions"
>
  <StripeElement
    type="payment"
    ...
  />
</StripeElements>
<template />

const elementsOptions = ref({
  clientSecret: 'grab_it_from_payment_intent',
  // https://stripe.com/docs/js/elements_object/create#stripe_elements-options
})

Screenshot


Dirty flag propagation

Using touch on nested field does not propagate to top level dirty flag unless all nested fields are dirty.

It can be tested on https://monterail.github.io/vuelidate/#sub-data-nesting
form$dirty is true if both nestedA.$dirty and nestedB.$dirty are true.
This is true for flat data too.

This may be the correct behavior for validate on save scenario so code may depend on this behavior.

That is indeed intended. $touch propagates to children, not the other way around.

The problem here is that there doesn't seem to be an easy way to check if any child is $dirty or $invalid.  I would expect that $v.$dirty would return true if any children are dirty. This is clearly not the case.

This is probably going to be resolved by #84


Issue HTML helper after use requestAction

Hi!
I have a problem. Can you help me to fix it

In Controller a_controller, call requestAction( b_controller/act)
In View call $this->Html->url( array( 'action' => 'otherAction'));

In origin Cake 1.3.21, it return url "/a_controller/otherAction/"
In this case: It return url: "/b_controller/otherAction"

Could you please help!
Thanks
Huynh Le

I could verify the changed behavior.
The cause was that the $this variable was set somewhere, and was used in the static method Router::url().
This worked in the PHP 5 version somehow, but after the port to PHP 7 $this was not set to the same value anymore.
I hope I didn't miss a deeper root cause.


Latin, non-ascii initials

Describe the issue
https://datatracker.ietf.org/doc/html/draft-ietf-opsec-probe-attribution renders the first author, Éric, with an E and not an É.
There seems to be some sort of transliteration going on here, but that is probably unnecessary.
This issue might be moot if we move to including names in full, as we should.  However, if we do nothing about that, then fixing this might help.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Describe the issue
https://datatracker.ietf.org/doc/html/draft-ietf-opsec-probe-attribution renders the first author, Éric, with an E and not an É.

As requested by the authors:
author:
 -
    ins: E. Vyncke
    name: Éric Vyncke

    <author initials="E." surname="Vyncke" fullname="Éric Vyncke">

Fix needed in document source.

Closing this as not an xml2rfc issue.


Update to use GitHub's environment files

https://github.com/oracle-actions/setup-java/blob/2e3562c72585cd036e42706589f272ed8d08f24d/src/Download.java#L221

Action and workflow authors who are using save-state or set-output via stdout should update to use the new environment files.

https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Are you suggesting to modify this line with the following ?
System.out.printf("echo \"%s=%s\" >> $GITHUB_OUTPUT%n", name, value);

Or do you see other changes that are necessary ?
(I am interested in taking this issue)

Yes, as far as I can tell, it's the only line that requires a change:

Your proposed change woudn't work on Windows (cmd.exe nor PowerShell), would it?
Perhaps using NIO's Files.write[String]() is platform-agnostic solution. Something like:
var output = Path.of(System.env...);
Files.write(
    output,
    List.of(name + "=" + value),
    StandardCharsets.UTF_8,
    StandardOpenOption.CREATE,
    StandardOpenOption.APPEND,
    StandardOpenOption.WRITE);

Before you hack away and create a pull request, I first need to clarify whether and how contributions can be submitted.

Starting today runner version 2.298.2 will begin to warn you if you use the save-state or set-output commands via stdout.

Warnings are now in place.

@derlin I am taking care of this issue because I want to get rid of the warnings as soon as possible. That's why I removed the good first issue label and filed:

#46

Outputs are empty?!

Fixed by https://github.com/oracle-actions/setup-java/commit/e81158a0e3112b4b04c18bbdcd2b276a6df82436


Kong 2.3.3 image contains CVE-2021-3450

Aquasec reports that Kong 2.3.3 image contains CVE-2021-3450
An updated Kong 2.3.x image is required to fix the openssl 1.1.1h defect identified in Kong 2.3.3 image.

@cmcbugg
Kong Gateway is not affected by CVE-2021-3450, since we don't use X509_V_FLAG_X509_STRICT in the codebase.
We have bumped the version of OpenSSL to 1.1.1k as a matter of good hygiene, see: https://github.com/Kong/kong/pull/6966

Is the OpenSSL version being updated to 1.1.1k on the 2.3.x branch?

@cmcbugg We will not backport the OpenSSL 1.1.1k bump to older versions like 2.3.x, since it's a no-op for us.


viewInsets.bottom will change while request next focus which should not

Steps to reproduce
If you put several TextField in a column of something else, while you request next focus on iOS, you will see that the viewInsets.bottom will change super fast between keyboard's final height and another.
Expected results
no matter how I request focus, the viewInsets.bottom should not be changed cause the keyboard never closed or attend to be.
Actual results
the viewInsets.bottom will change super fast between keyboard's final height and another.
 GOColor.ff191919 : GOColor.ff8a6d1f,
                  onTap: () => changePassword(ref),
                  child: const Center(
                    child: Text(
                      'CHANGE PASSWORD',
                      style: TextStyle(color: GOColor.white),
                    ),
                  ),
                );
              }),
              const GOBottomSpacer(),
            ],
          ),
        ),
      ],
    )

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.22.3, on macOS 14.5 23F79 darwin-arm64, locale zh-Hans-CN)
    • Flutter version 3.22.3 on channel stable at /Users/cals/SDK/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (4 weeks ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/cals/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)

[✓] VS Code (version 1.92.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.94.0

[✓] Connected device (4 available)
    • iPhone 15 Pro Max (mobile)      • 5A521D79-B614-46BB-8D1A-012B32828E36 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-17-5 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 14.5 23F79
      darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 14.5 23F79
      darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome
      127.0.6533.100

[✓] Network resources
    • All expected network resources are available.

• No issues found!

GOColor

@CalsRanna Are you using any third party implementation ? as I see GO related custom implementation in the code you shared. If yes, please remove it and check if you still get same behavior as reported and then provide us updated complete runnable code sample. Also, provide a short video of the behavior you are seeing so we have clear idea of what the issue you are reporting.

Please provide complete runnable code sample (ex: main.dart) that we can directly use to verify this further.


Update Documentation/Wiki

Update documentation based on recent activity. Most notably being:

Issue/PR
Description

#49
Parallel Test Execution/Discovery

#98
Working Directory

#99
Environment Variables

#107
Standard Output/Error Configuration

#108
Test Inclusion/Exclusion Configuration

#109
--list_content Support

Documentation updated on our fork: https://github.com/netspiri/vs-boost-unit-test-adapter/wiki

:+1:


Theme not updated in correct directory as specifiied by book.toml

https://github.com/zjp-CN/mdbook-theme/commit/df01754a32f259a9f001ea653c8e9de67c59c0b9 broke theme updates for me.
The theme now always seems to get installed into the project root ./theme directory.
My theme directory is actually in book/assets/theme
And is properly declared in the book.toml as:
[output.html]
theme          = "book/assets/theme"

Thanks for reporting! Fixed in v0.1.4 .


Add personal information

Edit AboutUs.md with personal information

Add personal image

Add portfolio file

Codecov Report

Merging #11 (567bce5) into master (b622516) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Fix #1.


Add missing server property and allow setImmediate in pluigns

Codecov Report

Merging #20 into master will increase coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master      #20     +/-   ##
=========================================
+ Coverage   55.55%   55.66%   +0.1%
=========================================
  Files           9        9
  Lines         423      424      +1
  Branches       85       85
=========================================
+ Hits          235      236      +1
  Misses        188      188

Impacted Files
Coverage Δ

lib/plugin.js
83.87% <100%> (+0.17%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8ad2811...64a75db. Read the comment docs.

LGTM. Just update Changes.md, merge & publish.

Ta - doesn't look like I've got permissions to npm publish though.   Username is 'smfreegard'.

When I try to add you to the module as a collaborator, npm says, user not found: smfreegard

Do you need to add me to the Haraka organization first?  - the user definitely exists:
smf@i7desktop:~/git/haraka-test-fixtures$ npm login
Username: smfreegard
Password:
Email: (this IS public<EMAIL_ADDRESS>Logged in as smfreegard on https://registry.npmjs.org/.

Weird. I've just been able to add you to the Haraka collabs on npm (which I wasn't able to do before), but when I try to do the same on haraka-plugin-fixtures (where the same 3 collabs (haraka, myself, and baudehlo exist) I get that user not found error.

Ok - can you npm publish this and it should cure the test failures on the other PRs I've done.   I've no idea what the problem with npm is.

$ npm publish
+<EMAIL_ADDRESS>
I still don't know what's up with npm. I've been able to add your profile to a couple other @haraka modules that I couldn't before. I'll try again in the future and maybe whatever this issue is will have cleared.


Change default ExtData to 2G in MAPL_CapGridComp/Get Rid of Build with ExtData2G logic

In MAPL_CapGridComp.F90 can we please change this line
call MAPL_GetResource(maplobj,use_extdata2g,"USE_EXTDATA2G:",default=.false.,_RC)

to this line
 call MAPL_GetResource(maplobj,use_extdata2g,"USE_EXTDATA2G:",default=.true.,_RC)

The fact is the model has been using with  USE_EXTDATA2G: .true. in its CAP.rc file FOR A LONG LONG LONG TIME.  ExtData1G is dead code and not maintained. No one should be running it by default in any future release of MAPL. The model relies on features not even in ExtData1G, so it would not work anyway, there is no rational reason to allow users to run ExtData1G, unless they have not converted, and as far as GEOS, the primary customer, there is no reason. Why would we want the default to be something that we KNOW won't work with the primary user. Also for my testing purposes, it would just be easier if this were the default. Like anytime want to reproduce a problem I take one of the test cases as a starting point and have to add USE_EXTDATA2G: .true. to the input files for ExtDataDriver.x even though 99% of the time, I'm doing something with ExtData2G, so this just slows me down. It's not like we are not giving users the option to use the old ExtData. This just reflects reality and makes things easier for me.
2nd in CMake and the code we have all these BUILD_WITH_EXTDATA2G macros. I think this was because we didn't want to introduce a YafYaml dependence. Now that ExtData2G is using ESMF_HConfig, the reason to have this conditional has gone away, right?
@mathomp4 what do you think?

I'll make branch with this change and update the tests and make a PR.

closing this


Publish a new release supporting php 8.2 syntax?

See #370 #374 #375 and #382 , in addition to changes that were already merged to support php 8.2 syntax or other improvements

@roblourens fyi - I think the latest main commit should be fine to release, though https://github.com/microsoft/tolerant-php-parser/pull/382 would be nice to have

@roblourens - pinging again about publishing a release since it's easy to lose track of notifications on the weekend.
PHP 8.2.0RC2 is out https://www.php.net/ - https://wiki.php.net/todo/php82 will be out in November

I got to this last - everything I merged should be fine to go out in a 0.1.2 right?

Yes - everything merged is fine to go out in a 0.1.2 release

Tests of tolerant-php-parser pass in ci in 8.2.0beta2 and locally in a recent 8.2-dev build
Tests of Phan pass in 8.2/7.4 (e.g. the component that converts Microsoft\PhpParser to php-ast ast\Node with/without error tolerance) after replacing vendor/microsoft/tolerant-php-parser/src with a35ec03ed3d76940e45955acc98fc8c43f6da081
I looked over recent changes and they all looked good

I realized one thing - the way HaltCompilerStatement is parsed could use some work for easily inferring __HALT_COMPILER_OFFSET__ and testing edge cases - https://github.com/microsoft/tolerant-php-parser/pull/389
If a release is created before that I can just keep the property name of semicolon for ; or ?> or ?>\n

I realized one thing - the way HaltCompilerStatement is parsed could use some work for easily inferring HALT_COMPILER_OFFSET and testing edge cases - https://github.com/microsoft/tolerant-php-parser/pull/389

But other than #389, there were no issues with extracting the needed information from new parsed nodes (e.g. exact same computed offset for AST_HALT_COMPILER) when I started using tolerant-php-parser main to test. https://github.com/phan/phan/pull/4730 has tests passing for converting them to ast\Node with the same fields

@roblourens #389 continues to be the only PR with fixes I'd want before creating a release with the other changes - could you review that?

If you want to keep the existing semicolon name instead then feel free to make that change

https://github.com/microsoft/tolerant-php-parser/pull/391 should also be included to fix a regression

Done


Implement hardware offloading override

Implement hardware offloading override for selected interfaces. Allow to have global flag to turn-off all overrides. Also clean-up how hardware offloading is set/unset, since flags are idempotent.  #1431

@fichtner thanks for review. I agree on all counts, will first retract the interface.lib.inc changes and submit as a separate PR. Then will change the override itself to per option for each interface. Thanks.

I will close this PR for now, since there is a need to fix the way flags are assigned to the interface, and the GUI in front of it.


Allow schema loading with an environment variable

Allow tests to use alternate schema paths via an environment variable in
riak_core.
Bump Clique to 0.2.4 so that schema loading is done explicitly by the
caller and not automatically during application startup.

Dup of #692

Dup of #692


Enhancement/configurable banners

This PR:

resolves #142.

haha i was just about to ask you about this. will review shortly!!
On Mon, Aug 7, 2017 at 13:51 Jason McGibbon<EMAIL_ADDRESS>wrote:

@wisygig https://github.com/wisygig requested your review on:
massgo/league#146 https://github.com/massgo/league/pull/146
Enhancement/configurable banners.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/massgo/league/pull/146#event-1196118787, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ADi8se3thXSDx26ROlSxG5fev7S0_XEvks5sV3jMgaJpZM4Ov71m
.

Rebased onto master


pkgname wildcard and version can not be mixed

Just trying to install nodejs 4.3.1 on a fresh debian 8.3 machine and I'm getting this error;
failed: [<IP_ADDRESS>] => {"failed": true}
msg: pkgname wildcard and version can not be mixed

FATAL: all hosts have already failed -- aborting

Here is my playbook
- hosts: webservers
  user: map7
  sudo: yes
  roles:
    - role: williamyeh.nodejs
      nodejs_version: "4"

Thanks.
I think the issue was solved in https://github.com/William-Yeh/ansible-nodejs/commit/306f0efce7ef2198e282d591fdce0f2dbfcd3a5f

I just updated to the latest version of your nodejs package and now when I run my playbook I get
ERROR: block is not a legal parameter in an Ansible task or handler
Here is my playbook.
- hosts: webservers
  user: map7
  become: yes
  roles:
    - williamyeh.nodejs

  vars:
    nodejs_version: 4.0

I'm using ansible 1.9.4 maybe that is the problem.

Yes, I'm gradually updating all my roles with Ansible 2.0 syntax such as block.


Add version: Minio.Server version 2024.01.01

- [X] Have you signed the Contributor License Agreement?

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

Hi @Trenly,
This PR appears to modify multiple PRs - is this just an artifact of how GitHub displays this PR?

Hi @Trenly,
This PR appears to modify multiple manifests- is this just an artifact of how GitHub displays this PR?

Yes; If it did modify multiple then it wouldn’t pass the pipelines

Hi @Trenly,

This PR's version number 2024.1.1 has 3 parameters (sets of numbers between dots - major, minor, etc), which is greater than the current manifest's version latest, which has 1 parameters.

Is this intentional?
(Automated response - build 510)


refactor(branch-stage): apply BBN stages as workers finish

The main purpose here is to improve the encapsulation of the branch stage to its own module.
However, this'll also serve as a small optimization of a couple milliseconds. Rather than collecting all the changes into a bigger BTreeMap and iterating it at the end, we can remove some CPU load from the critical path.
I had to comment out the update quickcheck tests for a moment and will fix them after this stack is completed.

#525  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rphmeier and the rest of your teammates on  Graphite

Merge activity

Oct 31, 5:25 PM EDT: A user started a stack merge that includes this pull request via Graphite.


Locally supressing code analysis warnings

From<EMAIL_ADDRESS>on 2011-05-15T03:30:58Z
Consider the following code
exec('anotherFile.py') #defines a variable called v
v += 10 # pyflake marks this line: "undefined name v"

In this code a variable "v" was created by an exec call. Subsequent calls to "v" invoke code analysis error messages. Would be nice to be able to do something like this (inspired by the syntax suggested here: http://djangosnippets.org/snippets/1762/ ):
exec('anotherFile.py')
v += 10 #pyflake:ignore

Original issue: http://code.google.com/p/spyderlib/issues/detail?id=655

From ccordoba12 on 2011-05-15T08:15:53Z
It seems and interesting enhancement.
Labels: -Type-Defect Type-Enhancement

From pierre.raybaut on 2011-05-16T01:32:35Z
Labels: Cat-Editor

From pierre.raybaut on 2011-05-22T09:46:22Z
Thanks for this excellent suggestion.
Status: Accepted

From pierre.raybaut on 2011-05-22T09:47:37Z
This issue was closed by revision 92f857dafd4d .

From pierre.raybaut on 2011-05-22T09:47:26Z
This issue was closed by revision 60d4f14c8805 .
Status: Fixed

From pierre.raybaut on 2011-05-23T09:58:14Z
Of course, see for example: https://code.google.com/p/spyderlib/source/browse/spyderlib/widgets/dicteditor.py#14

From<EMAIL_ADDRESS>on 2011-05-23T09:31:40Z
Is there a similar construct for pylint?


incorrect compile error

I have a property defined as:
/**
 * @type {test.DisplayObject|Array<test.DisplayObject>|Array<test.TableRow>}
 */
this.value;

but the compiler seems to be confused about it:
found   : (Array|null|test.DisplayObject)
required: (null|test.DisplayObject)
        var /** test.DisplayObject|Array<test.DisplayObject>|Array<test.TableRow> */ value = new test.Result().value;
                                                                                ^

It seems to have collapsed the typed arrays to Array for the found value and the arrays are completely missing from the variable's type.
I have created a gist that reproduces the problem.

I just pushed a new version of the debugger service, and I'm not seeing this. Maybe you're on an older compiler version? http://closure-compiler-debugger.appspot.com/#input0%3D%252F**%250A%2520*%2520%2540constructor%250A%2520*%2520%2540export%250A%2520*%252F%250Avar%2520test%2520%253D%2520function()%2520%257B%257D%253B%250A%2520%250Atest.prototype.doTest1%2520%253D%2520function()%2520%257B%250A%2509var%2520%252F**%2520test.DisplayObject%257CArray%253Ctest.DisplayObject%253E%257CArray%253Ctest.TableRow%253E%2520*%252F%2520value%2520%253D%2520new%2520test.Result().value%253B%250A%257D%253B%250A%250A%252F**%250A%2520*%2520%2540constructor%250A%2520*%2520%2540struct%250A%2520*%252F%250Atest.DisplayObject%2520%253D%2520function()%2520%257B%257D%253B%250A%2520%250A%252F**%250A%2520*%2520%2540constructor%250A%2520*%2520%2540struct%250A%2520*%252F%250Atest.TableRow%2520%253D%2520function()%2520%257B%257D%253B%250A%2520%250A%252F**%250A%2520*%2520%2540constructor%250A%2520*%2520%2540struct%250A%2520*%252F%250Atest.Result%2520%253D%2520function()%2520%257B%250A%2509%252F**%250A%2509%2520*%2520%2540type%2520%257Btest.DisplayObject%257CArray%253Ctest.DisplayObject%253E%257CArray%253Ctest.TableRow%253E%257D%250A%2509%2520*%252F%250A%2509this.value%253B%250A%257D%253B%26input1%26conformanceConfig%26externs%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26MISSING_PROPERTIES%3D1

I'm using the latest version of the compiler (v20150505).
I'm not sure what to do with your link - I tried pasting my reproducer code into the text boxes but it says
input0:1: ERROR - variable goog is undeclared
goog.provide('test');
^

My link is the same, just without the goog.provide (since the debugger service doesn't have the Closure Library built in)
What is the full set of command line flags you're using?

java -jar compiler.jar ^
--js=_closure-library ^
--js=c:\temp\test.js ^
--compilation_level=ADVANCED_OPTIMIZATIONS ^
--warning_level=VERBOSE ^
--jscomp_error=accessControls ^
--jscomp_error=checkRegExp ^
--jscomp_error=const ^
--jscomp_error=constantProperty ^
--jscomp_error=strictModuleDepCheck ^
--jscomp_error=visibility ^
--jscomp_error=missingProperties ^
--only_closure_dependencies ^
--closure_entry_point=test ^
--js_output_file=c:\temp\test_compiled.js ^
--generate_exports

@ghosttie Are you still getting that error?
While I couldn't reproduce the typing error, I'm getting a crash shown in #1393.

Can't reproduce the original error in the latest release (v20151216), closing this.

No, I can no longer reproduce it, thanks.


Add automatic support for S3 SSE

This checks to see if your bucket policies prevent the creation of unencrypted
files. If so, future PutObject requests will use the SSE request header.
This currently assumes you are using AWS KMS-managed encryption keys.

What about something like this:  https://github.com/frugalmechanic/fm-sbt-s3-resolver/commit/24b57bf3576be36ec4497fcf95d069dfbd3f13e2
I think it accomplishes the same thing but avoids the creation of the tmp object.  It basically just tries the PUT as normal and if that fails with a 403 status code then retries with SSE.  If the SSE PUT succeeds then it caches that for the next PUT to the bucket.
Thoughts?
I have my sse branch (https://github.com/frugalmechanic/fm-sbt-s3-resolver/tree/sse) currently published as 0.11.0-SNAPSHOT to Sonatype.

Looks good! Thanks

This has been published as v0.11.0